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
Launch Manage Students Table! #20981
Conversation
0ad9025
to
a8530e4
Compare
%th{colspan: "4", style: "padding-top:5px;padding-bottom:5px"} | ||
%th.manage-th{style: "padding-top:5px;padding-bottom:5px; height:45px"} | ||
%button.btn.btn-primary{"ng-click" => "save(section.students)", "ng-show" => "editingAny(section.students)", style: "float:left", "ng-disabled" => "allForm.$invalid"}= I18n.t('dashboard_action_save_all') | ||
|
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.
🎉!!!!
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.
Awesome! 👏
.managebuttons{style: "float:right; margin-bottom: 9px;", "ng-if" => "tab == 'manage' && teacher_managed_section(section)"} | ||
%span{"ng-if" => "section.login_type !== 'email'"} | ||
%button.btn.btn-white{"ng-click" => "new_student()", "ng-disabled" => "bulk_import.editing"}= I18n.t('dashboard_students_add_student') | ||
%button.btn.btn-white{"ng-click" => "bulk_import.editing = true", "ng-disabled" => "bulk_import.editing"}= I18n.t('dashboard_students_add_students') |
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.
Can we remove the i18n strings dashboard_students_add_student
and dashboard_students_add_students
?
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.
These are all gsheet controlled, so I'll remove once this PR gets merged.
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.
Removed all unused strings from gsheet 👍
%col{width: "250"}/ | ||
%tr | ||
%th.manage-th | ||
%a{href: "", "ng-click" => "order = 'name'"}= I18n.t('dashboard_students_name') |
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.
Likewise any strings in here? (Not sure which are still used elsewhere, there are a lot).
Got approval from Poorva. |
I'll merge this once I get the go ahead from Poorva.