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

feat(curriculum): add English curriculum block 8.3 expressing agreement or disagreement #54352

Merged
merged 8 commits into from
May 7, 2024

Conversation

miyaliu666
Copy link
Member

Checklist:

@miyaliu666 miyaliu666 requested a review from a team as a code owner April 11, 2024 11:59
@github-actions github-actions bot added scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory. scope: i18n language translation/internationalization. Often combined with language type label labels Apr 11, 2024
@miyaliu666 miyaliu666 added the new english course English Curriculum label Apr 11, 2024
@naomi-lgbt naomi-lgbt added the status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending. label Apr 25, 2024
Copy link
Member

@moT01 moT01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good 🎉 I pushed a few minor grammar fixes.

A couple other concerns:

Task 16: the answers are in backticks and won’t be translated. It feels like they shouldn’t be in backticks, but I'm not sure.

Tasks 27, 28, and 31 have quotes in the backticks - e.g. `"I weakly believe in our team's weaknesses."` I'm not sure if we need the quotes. I would vote no.

@moT01 moT01 merged commit 3836e4b into freeCodeCamp:main May 7, 2024
22 checks passed
@miyaliu666 miyaliu666 deleted the feat-8.3 branch May 9, 2024 02:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new english course English Curriculum scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory. scope: i18n language translation/internationalization. Often combined with language type label status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants