chore: rename APIs and Microservices to include "Backend"#42515
chore: rename APIs and Microservices to include "Backend"#42515ShaunSHamilton merged 21 commits intofreeCodeCamp:wip/certification-renamefrom heysujal:main
Conversation
Corrected a grammar mistake by removing a comma.
raisedadead
left a comment
There was a problem hiding this comment.
Hi @heysujal
Thanks for the PR.
We recommend reading & following the contributing guide.
The pull-request template presented to you when opening a PR is there for a reason and we insist you follow it :)
For instance, the main branch of your fork will be soiled if and when we land this. While you can fix it up, it's non-trivial and will be a bit frustrating. We cover all of these pitfalls in our contributing guide.
Finally as for the PR. The changes seem good, please ensure the tests are passing.
Thanks again.
I will keep this in mind from now on. Thanks |
|
With this current change, old challenge links will 404: - /learn/apis-and-microservices/apis-and-microservices-projects/file-metadata-microservice
+ /learn/back-end-development-and-apis/back-end-development-and-apis-projects/file-metadata-microserviceMrugesh mentioned the possibility to re-route: https://github.com/freeCodeCamp/client-config/blob/master/serve.json However, this change can be reverted. CC @ojeytonwilliams Also, if this re-route is accepted, I will change the links in the READMEs throughout the codebase. By 'I', I mean create an issue, and get help |
|
Note to self: Naming should be |
|
Should it? Front end doesn't include a hyphen. |
According to us, yes... https://contribute.freecodecamp.org/#/how-to-work-on-coding-challenges#:~:text=Front-end%20development |
I've spent enough time working on the docs that I should know this.... |
|
@heysujal Thank you, for taking the time to contribute on this. I have just merged this PR into a work-in-progress branch on |
* fix typo * fix typo * undo change * Corrected grammar mistake Corrected a grammar mistake by removing a comma. * change APIs and Microservices cert title * update title * Change APIs and Microservices certi title * Update translations.json * update title * feat(curriculum): rename apis and microservices cert * rename folder structure * rename certificate * rename learn Markdown * apis-and-microservices -> back-end-development-and-apis * update backend meta * update i18n langs and cypress test Co-authored-by: Shaun Hamilton <shauhami020@gmail.com>
* chore: rename APIs and Microservices to include "Backend" (#42515) * fix typo * fix typo * undo change * Corrected grammar mistake Corrected a grammar mistake by removing a comma. * change APIs and Microservices cert title * update title * Change APIs and Microservices certi title * Update translations.json * update title * feat(curriculum): rename apis and microservices cert * rename folder structure * rename certificate * rename learn Markdown * apis-and-microservices -> back-end-development-and-apis * update backend meta * update i18n langs and cypress test Co-authored-by: Shaun Hamilton <shauhami020@gmail.com> * fix: add development to front-end libraries (#42512) * fix: added-the-word-Development-to-front-end-libraries * fix/added-the-word-Development-to-front-end-libraries * fix/added-word-development-to-front-end-libraries-in-other-related-files * fix/added-the-word-Development-to-front-end-and-all-related-files * fix/removed-typos-from-last-commit-in-index.md * fix/reverted-changes-that-i-made-to-dependecies * fix/removed xvfg * fix/reverted changes that i made to package.json * remove unwanted changes * front-end-development-libraries changes * rename backend certSlug and README * update i18n folder names and keys * test: add legacy path redirect tests This uses serve.json from the client-config repo, since we currently use that in production * fix: create public dir before moving serve.json * fix: add missing script * refactor: collect redirect tests * test: convert to cy.location for stricter tests * rename certificate folder to 00-certificates * change crowdin config to recognise new certificates location * allow translations to be used Co-authored-by: Nicholas Carrigan (he/him) <nhcarrigan@gmail.com> * add forwards slashes to path redirects * fix cypress path tests again * plese cypress * fix: test different challenge Okay so I literally have no idea why this one particular challenge fails in Cypress Firefox ONLY. Tom and I paired and spun a full build instance and confirmed in Firefox the page loads and redirects as expected. Changing to another bootstrap challenge passes Cypress firefox locally. Absolutely boggled by this. AAAAAAAAAAAAAAA * fix: separate the test Okay apparently the test does not work unless we separate it into a different `it` statement. >:( >:( >:( >:( Co-authored-by: Sujal Gupta <55016909+heysujal@users.noreply.github.com> Co-authored-by: Noor Fakhry <65724923+NoorFakhry@users.noreply.github.com> Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com> Co-authored-by: Nicholas Carrigan (he/him) <nhcarrigan@gmail.com>
I have renamed "APIs and Microservices" to "Back End Development and APIs" and "Back End Development and APIs Certification" accordingly.
Related #42444