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
Manage students table: reset password for secret words and images #20343
Conversation
editing_password: true, | ||
}), | ||
}).done((data) => { | ||
console.log(data); |
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.
Do we usually keep these console.logs in or were you just using them to debug?
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.
oops good catch
} else if (this.props.loginType === SectionLoginType.word) { | ||
this.props.setSecretWords(this.props.id, data.secret_words); | ||
} | ||
}).fail((jqXhr, status) => { |
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.
what's jqXhr?
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.
I also had to look this up - it's the short name for the object we get back when we make an ajax request: http://api.jquery.com/jquery.ajax/
userType: 'student', | ||
age: 17, | ||
gender: 'f', | ||
loginType: 'picture', |
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.
tiny ? : are picture logins mostly for younger kids?
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.
younger students definitely use it almost exclusively, but some ms/hs teachers prefer it because its easier for students to manage
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.
A couple little questions for my own edification. Hooray for getting this working!
ManageStudentsTable is part of the teacher dashboard that is behind an experiment
manageStudents
, so this change won't be visible to users in production.This allows teachers to reset their students picture and word passwords from the table.
Redux tests included.