Skip to content
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

Fix "Koren Kii Error" links #121

Merged

Conversation

hyperupcall
Copy link
Contributor

@hyperupcall hyperupcall commented Nov 4, 2024

Description

When browsing this wonderful site, I noticed that these links were out of date. Didn't notice a CONTRIBUTING.md or similar so not exactly sure how project is built (with Crowdin and all, and not many PRs since #117 landed), so I just did a recursive-directory find & replace of the slug. Things linked properly when running npm run docs:dev (in addition to docs:build and docs:preview). If you prefer a different format / things done differently, I'd be happy to fix things up in accordance

@hyperupcall hyperupcall changed the title Fix "Koren Kii Errpr" links Fix "Koren Kii Error" links Nov 4, 2024
@playerjmr
Copy link
Collaborator

TBH the title itself seems confusing too as "Korean Kii" is the name of the homebrew application for removing the "Korean Key" which causes the error 003 brick in the first place. I think I ought to just change the header altogether

@playerjmr
Copy link
Collaborator

also, please dont make commits to other translations, as that will cause conflicts with crowdin.

@hyperupcall hyperupcall force-pushed the hyperupcall-fix-korean-kii-link branch from e6260ba to 61a18b1 Compare November 12, 2024 21:36
@hyperupcall
Copy link
Contributor Author

hyperupcall commented Nov 12, 2024

@playerjmr Agreed, if that is the case, then "Korean Key" may not be the best title. I've updated the PR to only change the base markdown files, feel free to merge or close depending on if the title is good

@playerjmr
Copy link
Collaborator

Yeah this is fine for now, The header change is a whole different project I dont want to deal with rn especially since im starting a new job tomorrow.

@playerjmr playerjmr merged commit bf500a7 into hacks-guide:master Nov 13, 2024
1 check passed
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