v2.5.0 #322
Merged
v2.5.0 #322
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
* add new handler to get group registration * remove scripts that will not be used * move getting registration data to its own router * refactor api endpoints * fix tests * fix another test * improve error message * remove migration * merge migrations
Co-authored-by: Martin Benedikt Busch <[email protected]>
…alculating the plurality score (#311) * update seed to create questions to group categories * fix tests * update vote service * fix existing tests * fix seed and update tests * test query group category ids * insert dummy uuid to allow IN in sql query * cleanup * remove unused var
* add migrations for secret * add create group type * update migrations for more control on user settings * create group handler * add join secret group handler * add group from category handler * add tests * add handler to router * fix rename issue
Prerelease 2.5.0
* remove secret from get category groups endpoint * fix typo on group categories router * add migration to show fields on group registration
* update join groups handler to support public groups * remove undefined category with seed * rename key from id to groupId in join group payload * update permissions on scripts * fix tests
* remove null values from usersToGroups * fix seed and tests * update seed * fix vote service tests * remove null from group handler
* add alerts table * add router for alerts * add title, description and link to alerts
* add migration for boolean registration field flags * add validation for user and group registration fields
* limit one registration per group * clean up logic * add authorization on update registration * fix authorization on an event level
…342) * adds better validation for fields and register field options in seed * make select field not required
diegoalzate
removed
DO NOT MERGE
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.
In Order to Merge this PR the following two conditions must be met:
1). A group category called
affiliationmust be created in production2). The groupCategoryId of affiliation must be inserted into the groups and usersToGroups table using the admin tool.
3). Set field display on group registration equal to true for existing fields
4). The groupCategoryId of affiliation in questions_to_categories