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: block team ORAs from ESG (for now) #1849
Conversation
Codecov Report
@@ Coverage Diff @@
## feature/esg-demo-mode #1849 +/- ##
========================================================
Coverage ? 94.99%
========================================================
Files ? 153
Lines ? 17246
Branches ? 1896
========================================================
Hits ? 16383
Misses ? 647
Partials ? 216
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* feat: block team ORAs from ESG (for now) * style: quality * test: test * chore: static * chore: version
* feat: stuff column should not be clickable on ESG enable test: update unit testing chore: update static chore: update translations chore: version bump * feat: add link from xblock staff area to ESG (#1839) Added a link to ESG demo when ESG is not enabled. Made the staff area buttons inline-block with a margin because they are now overflowing. I tried to mimic a Paragon Button in terms of coloring: used primary-500 for background, darkened by 7.5% on hover. * feat: block team ORAs from ESG (for now) (#1849) * feat: block team ORAs from ESG (for now) * style: quality * test: test * chore: static * chore: version * chore: Update version * chore: Update translations * chore: translations * chore: js versions Co-authored-by: Leangseu Kim <lkim@edx.org> Co-authored-by: Jansen Kantor <jkantor@edx.org>
TL;DR - block team ORAs from ESG (for now)
JIRA:AU-617 (Not exactly FOR this ticket, but relates to)
What changed?
Developer Checklist
Testing Instructions
In a course with both Team and Individual ORAs visit the Instructor Dashboard ORA page with the ESG enabled flag on and off. Click into the ORAs to see staff area.
Reviewer Checklist
Collectively, these should be completed by reviewers of this PR:
FYI: @openedx/content-aurora
Screenshots:
Instructor Dashboard (ESG Enabled)
Instructor Dashboard (ESG Disabled)
Team ORA Staff Area(ESG Enabled)
Team ORA Staff Area (ESG Disabled)
Staff ORA Staff Area (ESG Enabled) [Existing Behavior]
Staff ORA Staff Area (ESG Disabled) [Existing Behavior]