Skip to content

Conversation

@moyeah
Copy link
Contributor

@moyeah moyeah commented Sep 18, 2023

Description

Update Docusaurus

Included suggested comments from @hamza1311 on #3403.

Remove whitespace in tutorial.
Changes to be committed:
	modified:   website/docs/tutorial/index.mdx
	modified:   website/versioned_docs/version-0.20/getting-started/build-a-sample-app.mdx
	modified:   website/versioned_docs/version-0.20/tutorial/index.mdx
Changes to be committed:
	modified:   website/docs/migration-guides/yew/from-0_18_0-to-0_19_0.mdx
Changes to be committed:
	modified:   website/docs/getting-started/build-a-sample-app.mdx
	modified:   website/docs/tutorial/index.mdx
	modified:   website/versioned_docs/version-0.20/getting-started/build-a-sample-app.mdx
	modified:   website/versioned_docs/version-0.20/tutorial/index.mdx
Changes to be committed:
	new file:   package-lock.json
	modified:   website/docs/tutorial/index.mdx
	modified:   website/package-lock.json
	modified:   website/package.json
	modified:   website/versioned_docs/version-0.20/getting-started/build-a-sample-app.mdx
	modified:   package-lock.json
	modified:   package.json
@github-actions
Copy link

github-actions bot commented Sep 19, 2023

Visit the preview URL for this PR (updated for commit 86ec288):

https://yew-rs--pr3406-update-docusaurus-uckaksot.web.app

(expires Fri, 06 Oct 2023 15:22:47 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

cecton
cecton previously approved these changes Sep 23, 2023
Copy link
Contributor

@cecton cecton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

huh this PR is mixed with #3405 maybe on purpose? It probably doesn't matter at all

Copy link
Member

@ranile ranile left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you rebase/merge to include the commits from master?

…bsite

Changes to be committed:
	modified:   website/docs/tutorial/index.mdx
	modified:   website/versioned_docs/version-0.20/getting-started/build-a-sample-app.mdx
@moyeah moyeah mentioned this pull request Sep 23, 2023
@moyeah moyeah requested a review from ranile September 23, 2023 18:40
@ranile
Copy link
Member

ranile commented Sep 29, 2023

@moyeah I'm so sorry for repeatedly asking for changes but can you please ensure there are no conflicts and this PR only updates Docusaurus

@moyeah
Copy link
Contributor Author

moyeah commented Sep 29, 2023

@moyeah I'm so sorry for repeatedly asking for changes but can you please ensure there are no conflicts and this PR only updates Docusaurus

@hamza1311 no need for sorry. I will do it as soon I get a time slot to do it.

@ranile
Copy link
Member

ranile commented Sep 29, 2023

Also, as part of the update, can you also prepare for docusaurus v3

@moyeah
Copy link
Contributor Author

moyeah commented Sep 29, 2023

@hamza1311 I will close this PR.
I made different PR per subject:

@moyeah moyeah closed this Sep 29, 2023
@moyeah moyeah deleted the update-docusaurus branch September 29, 2023 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants