-
Notifications
You must be signed in to change notification settings - Fork 480
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
[ProgressV2] Add a coming soon
features banner
#58316
Conversation
coming soon
features banner
@@ -23,7 +23,7 @@ module.exports = { | |||
'selector-pseudo-class-no-unknown': [ | |||
true, | |||
{ | |||
ignorePseudoClasses: ['export'], | |||
ignorePseudoClasses: ['export', 'global'], |
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.
This is necessary in order to use the :global()
selector here. Which is necessary to modify styles of a child component in an external library. The alternative is to use .scss
without modules, which teacher tools is trying to move away from.
// expect(questionText).to.not.exist; | ||
// expect(shareMoreText).to.not.exist; | ||
// }); | ||
it('attempts reset and reload if error from createFeedback', async () => { |
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.
Thanks for adding this test!
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.
Looks good! Nice work on this!
Warning!!
The AP CSP Create Performance Task is in progress. The most critical dates are from April 3 - April 30, 2024. Please consider any risk introduced by this PR that could affect our students taking AP CSP. Code.org students taking AP CSP primarily use App Lab for their Create Task, however a small percent use Game Lab. Carefully consider whether your change has any risk of alterering, changing, or breaking anything in these two labs. Even small changes, such as a different button color, are considered significant during this time period. Reach out to the Student Learning team or Curriculum team for more details.
Adds a new banner with information about what features are coming soon to the progress page.
Only opens if:
Also fixes some bugs with the ProgressFeedback Banner.
Links
https://codedotorg.atlassian.net/browse/TEACH-1030
Testing story
Added unit tests for the new component and re-enabled unit tests for the existing component after bug fixes.
Deployment strategy
Follow-up work
Privacy
Security
Caching
PR Checklist: