Update link in assignment

[odilson-dev]

Because

This PR is about updating the link of Ressources in the asiignment section, it's improve the experience of students by guiding them directly to the section we want them to read.

This PR

• Update the link to the right section

Issue

Closes #XXXXX

Additional Information

Pull Request Requirements

• [x] I have thoroughly read and understand The Odin Project Contributing Guide
• [x] The title of this PR follows the location of change: brief description of change format, e.g. Intro to HTML and CSS lesson: Fix link text
• [x] The Because section summarizes the reason for this PR
• [x] The This PR section has a bullet point list describing the changes in this PR
• [x] If this PR addresses an open issue, it is linked in the Issue section
• [x] If any lesson files are included in this PR, they have been previewed with the Markdown preview tool to ensure it is formatted correctly
• [x] If any lesson files are included in this PR, they follow the Layout Style Guide

[CouchofTomato]

You also have some merge conflicts you'll need to address before we can merge this.

[odilson-dev]

Hi, i resolved merge conflicts, please let me know if any changes are requirded before merge.

[CouchofTomato]

Hi, i resolved merge conflicts, please let me know if any changes are requirded before merge.

It looks like merging in main may have reset one of the lines because I don't think it should be 21

[odilson-dev]

Hi, i resolved merge conflicts, please let me know if any changes are requirded before merge.

It looks like merging in main may have reset one of the lines because I don't think it should be 21

Just fixed that. Thanks

[CouchofTomato]

Hey @odilsoncode

Just wondering if you were still interested in finishing this PR up?

[odilson-dev]

Hi @CouchofTomato I thought this PR was already completed and merged but it seems that it is not. Please, let me know what other changes i've to make again so we can finish up.

[CouchofTomato]

Hey @odilsoncode

There are some merge conflicts. We can't merge until these are resolved.

[odilson-dev]

Hmmm... hi @CouchofTomato , i'm currently facing a bunch of errors on this PR, and i'm not sure where they come from. Any assistance please?

[thatblindgeye]

@odilsoncode You can ignore those errors, they're just from our new linter that is helping us flag things to help lessons comply with our layout style guide.

@CouchofTomato do the changes to the PR look good to you?

[MaoShizhong]

Is there stuff that's meant to be changed here? Currently the diff only shows a single change that's removing trailing whitespace, but otherwise doesn't seem to show any changes from the current lesson.

[CouchofTomato]

Yeah it was supposed to update the chapter links for the exercises because the rails guide changed them. I think I'll have to just edit it at some point.

[MaoShizhong]

@CouchofTomato

~~Interesting that this PR's latest state doesn't have those href changes (edit: ah, the last merge conflict resolution wiped the changes).~~

~~Currently, we have #28421 in review as part of the accessible link text epic for the Rails pathway, and that PR makes changes to the links in the Active Record Queries lesson. It doesn't change any of the hrefs, only the link text. Do you think it would be sensible to close this PR, and request href and section number changes for the relevant links in that PR instead?~~

~~From what I've gathered, the href ending in #group should be updated to #grouping, and some of the steps' section numbers need to be adjusted.~~

[MaoShizhong]

@CouchofTomato Just an update that the #group URL has been fixed to #grouping in #28421.

Regarding the required reading sections, could this PR be closed then a new issue opened up for contribution to handle that?

[CouchofTomato]

@MaoShizhong Yeah I'm ok with that

[MaoShizhong]

@CouchofTomato Then I'll leave the new issue with you, as you'll know best what needs to be changed.