-
Notifications
You must be signed in to change notification settings - Fork 2
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
Release 0.8.0 #747
Merged
Merged
Release 0.8.0 #747
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* add hooks for creating, updating and deleting UserList * add UserList dialogs to ManageListDialogs * fix existing tests * fix UserList validationSchema * wire up UpsertUserListDialog to Create new list button on UserListListingPage * wire up edit button on UserListDetailsPage * name property on RadioChoiceField in UpsertUserListDialog should be "privacy_level" not "published" * add tests for upsertUserList * add tests for deleting a UserList * change margins on form-row * add context menu to UserListListingPage items, wire up buttons and add tests * remove unit from 0px margin * remove extraneous 0 in margin * remove unnecessary omit * correct accidental renaming of manageListDialogs * use faker.helpers.arrayElement
This reverts commit 24c6ba8.
* adding initial models for user subscription * switching to manytomany for percolate query -> user rel
* add AddToUserList dialog and the supporting hooks * Add bookmark icon to LearningResource card and wire up to dialog * display proper dialog title if adding resource to UserList * PascalCase the privacy level of UserList * only show the bookmark button for authenticated users * set correct label on bookmark button * fix existing tests * add tests to AddListDialog.test.tsx for UserLists with supporting factories * only show add to learning path button on LearningResourceCard if authenticated and privileges are sufficient, add tests for UserList related code * use BookmarkBorderIcon for now
3a54edf
to
2065ddc
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Anastasia Beglova
Shankar Ambady
Chris Chudzicki
Carey P Gumaer
James Kachel
Matt Bertrand
Nathan Levesque
git
renovate[bot]