-
Notifications
You must be signed in to change notification settings - Fork 105
fix: support DOCUSAURUS_GIT_TOKEN and fix markdown/hooks configuration #826
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
@sheikhlimon is attempting to deploy a commit to the recode Team on Vercel. A member of the Team first needs to authorize it. |
Synced data from Linked IssuesLabels:
Assignees:
Milestones:
|
Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. The estimated time for response is 5–8 hrs. In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, a video recording, or an image of the update you made below, which helps speed up the review and assignment. If you have questions, reach out to LinkedIn. Your contributions are highly appreciated!😊 Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv. We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰 🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨ 📚Your perks for contribution to this community 👇🏻
If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊 |
30946f0
to
755d789
Compare
@Adez017 can you take a look at this now |
@iitzIrFan need your opinion on this |
i dont think we need this @sheikhlimon @iitzIrFan , the project is already working fine on all the locals and in production also . the main root issue is with the bangladesh server or internet . making this changes will make the redundant disruptions . |
@iitzIrFan @sanjay-kv , let me know your thoughts on this . |
What does local have to do with internet? I clearly explained this on my issue also it's not something I even faced. Someone else was having this issue. I even mentioned the link for that comment on my issue. Note
That's all. I think this is gonna clear the confusion and having a fallback is definitely not redundant. |
ok @sheikhlimon , by making any changes on the code base directly affect the production and in my opinion its working fine with most of the people . let me involve @sanjay-kv in this conversation |
755d789
to
0133865
Compare
sure and if you have time have a look at this https://claude.ai/share/5e39a70e-818d-4e31-8508-b4ce7b5d565c |
Description
Fixes #825
Type of Change
Changes Made
gitToken
onBrokenMarkdownLinks
configurationhooks
object insidecustomFields
.onBrokenMarkdownLinks
.Dependencies
Checklist
npm run build
and attached screenshot(s) in this PR.