-
Notifications
You must be signed in to change notification settings - Fork 36
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
[CHORE] [MER-0000] Integrate v0.28.10 #5125
Conversation
…have finished [BUG FIX] [MER-3771] Delay submission to ensure all save state calls have finished
* account for unordered pages * use page_context instead of current_page
* directly provide atempt guid * Auto format --------- Co-authored-by: darrensiegel <darrensiegel@users.noreply.github.com>
* convert numeric to text input * cleanup * Auto format * change regex to allow 300. and .300 as valid numbers * allow same numbers in answer authoring as in delivery --------- Co-authored-by: darrensiegel <darrensiegel@users.noreply.github.com> Co-authored-by: Anders Weinstein <andersw@cs.cmu.edu>
…ls in certain cases (#5108) * fix an issue where account creation was not properly filtering out LTI accounts * remove redundant clause * Auto format --------- Co-authored-by: eliknebel <eliknebel@users.noreply.github.com>
️✅ There are no secrets present in this pull request anymore.If these secrets were true positive and are still valid, we highly recommend you to revoke them. 🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request. |
2e6b2b5
to
ddea257
Compare
DO NOT SQUASH: Select "Merge Commit" when merging
I'm not sure how it got past CI, but it seems one of the hotfix changes broke a test (and also introduced a bug). Commit 19a16d9 fixes the issue. We'll likely need to make this change in the next v28 hotfix branch as well.