Skip to content

chore: add github repository link#35

Open
D-Lite wants to merge 3 commits into
chakra-ui:mainfrom
D-Lite:editgithub
Open

chore: add github repository link#35
D-Lite wants to merge 3 commits into
chakra-ui:mainfrom
D-Lite:editgithub

Conversation

@D-Lite

@D-Lite D-Lite commented May 30, 2023

Copy link
Copy Markdown
Contributor

edit

Add github repository link to the page footer

@vercel

vercel Bot commented May 30, 2023

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
vue-docs-next ✅ Ready (Inspect) Visit Preview Jun 26, 2023 8:55am

@Shyrro Shyrro left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey @D-Lite ,

Thanks for opening this PR!

If you look at the generated staging, all the Edit this page on Github links seem to be broken for some reason.

Also, the link shouldn't just be the repository, it should link to the right .md file. You can do this based on the url as we use file path routing for our documentation.

Let me know if you have anymore questions.

@D-Lite

D-Lite commented Jun 26, 2023

Copy link
Copy Markdown
Contributor Author

@Shyrro nuxt/nuxt#21340 This issue on the nuxt repository explained the error I was facing earlier. I had a workaround with computed properties as suggested on the issue.

Everything works fine now

@D-Lite

D-Lite commented Aug 6, 2023

Copy link
Copy Markdown
Contributor Author

@Shyrro @carwack , you might want to review this pr. It cuts across the entire application and a reproducible example exists already

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.

2 participants