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

Change course and library keys to use OpenedX, DemoCourse, etc #50

Merged
merged 5 commits into from
Feb 9, 2024

Conversation

kdmccormick
Copy link
Member

@kdmccormick kdmccormick commented Feb 8, 2024

Per convo in #49 (comment)

Before

  • course-v1:Axim+DemoX+demo_course
  • library-v1:Dummy+RESP_Q1

After

  • course-v1:OpenedX+DemoX+DemoCourse
  • library-v1:OpenedX+DemoRespiratoryQuestions

Copy link
Contributor

@sarina sarina left a comment

Choose a reason for hiding this comment

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

LooksgooD
OncetheorG
IschangedtO
OpenedX

@kdmccormick
Copy link
Member Author

image

@kdmccormick kdmccormick marked this pull request as draft February 8, 2024 19:59
per trademark guidelines
to match style of org and course number (OpenedX+DemoX)
@kdmccormick kdmccormick changed the title Change course and library keys to use Open_edX org Change keys: course-v1:OpenedX+DemoX+Demo_Course ; library-v1:OpenedX+DemoRespiratoryQuestions Feb 8, 2024
@kdmccormick kdmccormick changed the title Change keys: course-v1:OpenedX+DemoX+Demo_Course ; library-v1:OpenedX+DemoRespiratoryQuestions Change course and library keys to use OpenedX, DemoCourse, etc Feb 8, 2024
@kdmccormick
Copy link
Member Author

@sarina With us using OpenedX now, it was driving me crazy that everything besides demo_course was camel case, so I've pushed one last commit that changes the run from demo_course to DemoCourse.

So now we have:

Before

  • course-v1:Axim+DemoX+demo_course
  • library-v1:Dummy+RESP_Q1

After

  • course-v1:OpenedX+DemoX+DemoCourse
  • library-v1:OpenedX+DemoRespiratoryQuestions

Any objections?

@kdmccormick kdmccormick marked this pull request as ready for review February 8, 2024 21:32
@sarina
Copy link
Contributor

sarina commented Feb 8, 2024

@kdmccormick Overall I appreciate you as an engineer, so much. This attention to detail - and the need to make things right - exemplify this. Thank you for being you. Approved.

@kdmccormick kdmccormick self-assigned this Feb 8, 2024
@kdmccormick kdmccormick changed the base branch from master to tcril/depr-automation-workflow February 9, 2024 14:20
@kdmccormick kdmccormick changed the base branch from tcril/depr-automation-workflow to master February 9, 2024 14:20
@kdmccormick kdmccormick merged commit 3d08769 into master Feb 9, 2024
3 checks passed
@kdmccormick kdmccormick deleted the kdmccormick/keys branch February 9, 2024 14:24
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.

2 participants