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

Remove period at the end of lesson keys in csf #44721

Merged
merged 2 commits into from Feb 9, 2022
Merged

Conversation

dmcavoy
Copy link
Contributor

@dmcavoy dmcavoy commented Feb 8, 2022

Names of lessons were showing that they could not be translated for some lessons. After investigating we determined it was because of a period in the key for the lesson. This removes periods from names of existing lessons that were causing issues to unblock curriculum writers.

Thanks @bethanyaconnor for your help investigating this.

Links

Slack Convo

Testing story

  • Ran bundle exec rake build to make sure things build ok
  • Checked that these lessons no longer show the translation error message

Follow-up work

@dmcavoy dmcavoy requested review from bethanyaconnor and a team February 9, 2022 14:09
@@ -36,7 +36,7 @@
],
"lessons": [
{
"key": "Navigating the Code.org learning platform",
"key": "Navigating the Codeorg learning platform",
Copy link
Contributor

Choose a reason for hiding this comment

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

Will this also get messed up? Or is it only at the end of the key?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was also an issue. You can see Hannah linked to it in the slack thread

Copy link
Contributor

@megcrenshaw megcrenshaw left a comment

Choose a reason for hiding this comment

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

When I do a regex search in RubyMine for "lesson\.key": ".*\." I get matches from coursea, courseb, and coursec. If the period is also not allowed within the key itself, I'll re-search

Copy link
Contributor

@bethanyaconnor bethanyaconnor left a comment

Choose a reason for hiding this comment

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

Thanks for fixing these!

@dmcavoy
Copy link
Contributor Author

dmcavoy commented Feb 9, 2022

When I do a regex search in RubyMine for "lesson\.key": ".*\." I get matches from coursea, courseb, and coursec. If the period is also not allowed within the key itself, I'll re-search

Its not allowed anywhere in the key it looks like.

@dmcavoy
Copy link
Contributor Author

dmcavoy commented Feb 9, 2022

@megcrenshaw feel free to do another search and report back on findings. I'm going to merge this to unblock writers.

@dmcavoy dmcavoy merged commit 6fcca70 into staging Feb 9, 2022
@dmcavoy dmcavoy deleted the period-in-key branch February 9, 2022 17:07
@megcrenshaw
Copy link
Contributor

megcrenshaw commented Feb 9, 2022

@dmcavoy Thanks for the clarification! I'm also seeing it in cspassessment.script_json ("lesson.key": "NOTE: Deprecated. Teacher cannot see results. CSP Culminating Assessment") and equity-pd.script_json ("lesson.key": "Code.org Equity PD"). That's all I see. Searching "lesson\.key": ".*\..*" in RubyMine

@dmcavoy
Copy link
Contributor Author

dmcavoy commented Feb 9, 2022

@megcrenshaw thank you! Here is the follow up PR #44742

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.

None yet

3 participants