Promote from main to preprod #367
Merged
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
…lt Server Park" to "Server Parks"
ClientConfig reads RoleToServerParksMap from json file
Read role_to_serverparks_map_json from json file
Rename RoleToServerParksMap to RoleToServerParksMap
Server side RoleToServerParksDict config change
Client side RoleToServerParksDict config changes
fix: Add resolveJsonModule to tsconfig.server.json
…ed in separate ticket for adding more tests.
…erverpark-Access-To-CMA-Roles BLAIS5-4190 Add cma serverpark access to cma roles
* chore: fail_ci_if_error: true * refactor: Protected server API endpoint test passes successfully * feat: Add all tests to cover the code in /api/users POST endpoint * fix: Prevent disk I/O each time GET * request hits server * fix: Fix formatting: yarn eslint . --fix fix: Add typemoq as dependency * feat: Add all tests to cover the code in /api/users DELETE endpoint * feat: Add all tests to cover the code in /api/users GET endpoint * feat: Add all tests to cover the code in /api/roles GET endpoint * feat: Add all tests to cover the code in /api/change_password/:user GET endpoint * fix: Fix Linting with ESLINT: yarn eslint . --fix * fix: Load .env variables if current env is not PROD. * fix: Remove launch.json from source control as it should be dev local file. * feat: Add test for "external Blaise API changePassword endpoint rejects request with error message" * chore: No changes in yarn.lock file. * refactor: Removed redundant json map from client. We don't trust client side script, mapping happens on server side so only one json map file needed that is on the server side.
* Promote from main to preprod (#354) * refactor: Add DEFAULT to RoleToServerParksMap, renamed heading "Default Server Park" to "Server Parks" ClientConfig reads RoleToServerParksMap from json file Read role_to_serverparks_map_json from json file Rename RoleToServerParksMap to RoleToServerParksMap Server side RoleToServerParksDict config change Client side RoleToServerParksDict config changes fix: Add resolveJsonModule to tsconfig.server.json * fix: Updated snapshot for UTs on users component tests. * refactor: using blaise-api-node-client v1.1.0 * ci: Do not fail CI build if coverage is low. Coverage will be increased in separate ticket for adding more tests. * refactor: Add whitespace after comma in server parks list. --------- Co-authored-by: am-ons <[email protected]> Co-authored-by: Elinor Thorne <[email protected]> * Promote from main to preprod (#358) * refactor: Add DEFAULT to RoleToServerParksMap, renamed heading "Default Server Park" to "Server Parks" ClientConfig reads RoleToServerParksMap from json file Read role_to_serverparks_map_json from json file Rename RoleToServerParksMap to RoleToServerParksMap Server side RoleToServerParksDict config change Client side RoleToServerParksDict config changes fix: Add resolveJsonModule to tsconfig.server.json * fix: Updated snapshot for UTs on users component tests. * refactor: using blaise-api-node-client v1.1.0 * ci: Do not fail CI build if coverage is low. Coverage will be increased in separate ticket for adding more tests. * refactor: Add whitespace after comma in server parks list. * BLAIS5-4274 increase unit test coverage in bam (#356) * chore: fail_ci_if_error: true * refactor: Protected server API endpoint test passes successfully * feat: Add all tests to cover the code in /api/users POST endpoint * fix: Prevent disk I/O each time GET * request hits server * fix: Fix formatting: yarn eslint . --fix fix: Add typemoq as dependency * feat: Add all tests to cover the code in /api/users DELETE endpoint * feat: Add all tests to cover the code in /api/users GET endpoint * feat: Add all tests to cover the code in /api/roles GET endpoint * feat: Add all tests to cover the code in /api/change_password/:user GET endpoint * fix: Fix Linting with ESLINT: yarn eslint . --fix * fix: Load .env variables if current env is not PROD. * fix: Remove launch.json from source control as it should be dev local file. * feat: Add test for "external Blaise API changePassword endpoint rejects request with error message" * chore: No changes in yarn.lock file. * refactor: Removed redundant json map from client. We don't trust client side script, mapping happens on server side so only one json map file needed that is on the server side. --------- Co-authored-by: am-ons <[email protected]> Co-authored-by: Elinor Thorne <[email protected]> * refactor: Add DEFAULT to RoleToServerParksMap, renamed heading "Default Server Park" to "Server Parks" ClientConfig reads RoleToServerParksMap from json file Read role_to_serverparks_map_json from json file Rename RoleToServerParksMap to RoleToServerParksMap Server side RoleToServerParksDict config change Client side RoleToServerParksDict config changes fix: Add resolveJsonModule to tsconfig.server.json * refactor: using blaise-api-node-client v1.1.0 * BLAIS5-4274 increase unit test coverage in bam (#356) * chore: fail_ci_if_error: true * refactor: Protected server API endpoint test passes successfully * feat: Add all tests to cover the code in /api/users POST endpoint * fix: Prevent disk I/O each time GET * request hits server * fix: Fix formatting: yarn eslint . --fix fix: Add typemoq as dependency * feat: Add all tests to cover the code in /api/users DELETE endpoint * feat: Add all tests to cover the code in /api/users GET endpoint * feat: Add all tests to cover the code in /api/roles GET endpoint * feat: Add all tests to cover the code in /api/change_password/:user GET endpoint * fix: Fix Linting with ESLINT: yarn eslint . --fix * fix: Load .env variables if current env is not PROD. * fix: Remove launch.json from source control as it should be dev local file. * feat: Add test for "external Blaise API changePassword endpoint rejects request with error message" * chore: No changes in yarn.lock file. * refactor: Removed redundant json map from client. We don't trust client side script, mapping happens on server side so only one json map file needed that is on the server side. --------- Co-authored-by: Matthew Palmer <[email protected]> Co-authored-by: Elinor Thorne <[email protected]> Co-authored-by: Nik Webb <[email protected]> Co-authored-by: Alistair Grew <[email protected]> Co-authored-by: Richmond Rice <[email protected]> Co-authored-by: Sidra Javed <[email protected]> Co-authored-by: James Anthony Williams <[email protected]> Co-authored-by: motalm_ONS <[email protected]> Co-authored-by: Kristian T <[email protected]> Co-authored-by: Cal <[email protected]> Co-authored-by: BenjaminLambe_ONS <[email protected]> Co-authored-by: social-surveys-blaise-concourse <79969802+social-surveys-blaise-concourse@users.noreply.github.com>
…ch user with ability to change password (existing), user role (new)
…ofile, ensure current user is displayed first in the table
Promote from BLAIS5-3816 to main
# Conflicts: # server/server.ts # server/tests/index.test.ts # src/pages/users/UserUpload/NewUser.tsx # src/pages/users/UsersTable.tsx # src/pages/users/__snapshots__/Users.test.tsx.snap
Promote from BLAIS5-4274-extension to main
lambeb
approved these changes
Jul 9, 2024
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.
Auto generated by concourse:
Promote from main to preprod