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

Mailers: an outdated assignment reading numbers #29260

Open
2 of 3 tasks
MahdiRox opened this issue Dec 31, 2024 · 2 comments
Open
2 of 3 tasks

Mailers: an outdated assignment reading numbers #29260

MahdiRox opened this issue Dec 31, 2024 · 2 comments

Comments

@MahdiRox
Copy link

Checks

Describe your suggestion

in the assignment the sections that are ordered to be read are outdated and don't match the what's in the rails website anymore so they should be updated <3

Path

Ruby / Rails

Lesson Url

https://www.theodinproject.com/lessons/ruby-on-rails-mailers

(Optional) Discord Name

No response

(Optional) Additional Comments

No response

@JoshDevHub
Copy link
Contributor

JoshDevHub commented Jan 4, 2025

Thank you for making this issue @MahdiRox -- you're absolutely correct that this should be updated.

Acceptance Criteria

  • In the assignment section of the Rails Mailers lesson, update the section reading to match with the new layout of the information.
    • The first item can remain unchanged.
    • The second item needs to point to sections 6 and 7 of the mailers guide.
    • The third item needs to point to section 8.2 of the mailers guide.
    • The fourth item needs to point to sections 9 and 10 of the mailers guide.

Since the OP indicated they do not wish to work on this themselves, I'll open this up for any contributors who wish to submit a PR. Comment below, and a maintainer will assign you.

@MahdiRox
Copy link
Author

MahdiRox commented Jan 4, 2025

Yes im down to work on it

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

No branches or pull requests

2 participants