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

DTT (Staging > Test) [robo-dtt] #57874

Merged
merged 9 commits into from
Apr 8, 2024
Merged

DTT (Staging > Test) [robo-dtt] #57874

merged 9 commits into from
Apr 8, 2024

Conversation

deploy-code-org
Copy link
Contributor

No description provided.

stephenliang and others added 9 commits April 3, 2024 16:06
Previously, the login info only showed 'LMS' for a section. However,
this is ambiguous as the teacher could have different systems. Replace
this with the name of the actual LMS provider.
* zip search dropdown populating

* fixing continuous api calls

* made it so switching back to dropdown updates back to selectASchool option

* pr feedback

* test is tantalizingly close to passing

* test is passing

* one last bit of pr feedback

* adding statsig and amplitude events to new school association flow

* consolidating events calls
* Add message type for customizations, improve styling

* Finish formatting, refactor into helper for render

* Remove console logs, add new role

* Unrelated: fix height of compare models dialog

* Rename constant, simplify message removal
* create SelfPacedProgressTable.jsx

* update storybook

* clean up

* update comment on tableStyles.scss

* fix links

* fix prop types on storybook

* add ProgressBar component

* add rtl styles

* refactor progress bar

* update print certificate button copy

* update storybook
* add tabs

* use real tab values and style properly

* only show tabs if at least 2

* PR feedback

* remove unnecessary styling
feat(lms): show LMS name in login info column
@deploy-code-org deploy-code-org requested review from a team as code owners April 8, 2024 20:18
@deploy-code-org deploy-code-org merged commit 5f9be90 into test Apr 8, 2024
1 check passed
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

8 participants