-
-
Notifications
You must be signed in to change notification settings - Fork 3
fix(docs): read the docs link #52
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
Conversation
| { | ||
| text: "Read the Docs", | ||
| link: "/general/", | ||
| link: "/truetech/truecharts-common/", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not the same docs as before.
|
Also dont randomly check every checkbox. As you didnt do most of them |
|
Also does not fix the issue. As it only fixes one of a couple of hundred links. |
|
Unbelievable.
Ok, so how about TELL me what it was? Amazing how you spend time nitpick my change while your website is totally broken. Oh no! I checked too many boxes!! Deny this Pull Request immediately !!! Meanwhile the face of this org/project is a total mess. Your pirorites are wack. |
I suggest calming down. This is a Free and open source project. We appreciate prs if they are done in a proper way. So congratulations you wasted part of my time i have for this project. |
It isn't broken, it has a broken link on a specific page.
The whole website was restructured, so its not that simple.
Its not nitpicking, its normal review
No-one denied your PR because of it
The website has always been a bit hit-or-mis, yes.
For our maintainer time has been going into fixing actual functional issues with the things we make. |
Description
Self explanatory
⚒️ Fixes
⚙️ Type of change
🧪 How Has This Been Tested?
yes
📃 Notes:
✔️ Checklist:
feat(chart-name):,fix(chart-name):,chore(chart-name):,docs(chart-name):orfix(docs):