-
Notifications
You must be signed in to change notification settings - Fork 6
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
Vault explore #1163
Merged
robwild-affinitydigital
merged 11 commits into
PAR-2484-Unable-to-amend-legal-entities
from
vault-explore
Aug 27, 2024
Merged
Vault explore #1163
robwild-affinitydigital
merged 11 commits into
PAR-2484-Unable-to-amend-legal-entities
from
vault-explore
Aug 27, 2024
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
Removes app checklist on db_sanitise as config import etc are not needed
robwild-affinitydigital
merged commit Aug 27, 2024
ad07a54
into
PAR-2484-Unable-to-amend-legal-entities
6 of 7 checks passed
robwild-affinitydigital
added a commit
that referenced
this pull request
Oct 2, 2024
#1160) * PAR-2484 - Allow for Registered Organisations or Registered Charities who have changed entity type since registration to be added as a legal entity - Adds a new option to allow users to add an entity who validation fails CH or CC lookups but saves on APIs requests * PAR-2490 - Correcting logic on adding new legal entity to an organisation * PAR-2487: Removing check to check if partnership is active to allow for legal entities to be edited before they are approved * PAR-2484: Temp Removing config readonly * PAR-2484: Updating validation on ne legal entities * Vault explore (#1163) * Update config.yml (#1161) Removes app checklist on db_sanitise as config import etc are not needed * Update config.yml * Update config.yml * Update config.yml * Update config.yml * Update config.yml * Update push.local.sh * Update config.yml * PAR-2491 & PAR-2492: Updating CF scripts to use new user creds * PAR-2491: More user switching * PAR-2491: Using same db import as stage fr the test env * Updating build scripts for new user variables * PAR-2493: Validation on partnership legal entity * PAR-2487: Removing checks on amendments if partnership is not confirmed as legal entities can now be added so this is redundant and blocking * PAR-2484: Removing lookup for companies house and charities commission fields * PAR-2487: Updating Legal entities to allow for varied numbers * Removing logic for saving issue * PAR-2495: Sanitising DB of Par Person data & PAR-2484: Fixing Legal Entity adding on confirmed partnerships. Will need to remove the line ../vendor/bin/drush spp && \ once this is in master so the daily job does the sanitisation * Swapping CircleCI IP ranges in * Fixing build tasks for CircleCI IP ranges * Updates for db backup * PAR-2484: Updating legal entities revoke permnission * PAR-2484: Resolving aggressive user checks
robwild-affinitydigital
added a commit
that referenced
this pull request
Oct 23, 2024
* PAR-2498 sentry bot tasks clean up (#1169) * PAR-2498: Swapping operator for search_api opensearch valid condition * PAR-2498: Suppressing user redirect warning as an info log message * PAR-2498: Adding logic to ensure .well-known/traffic-advice is served and not 404 * PAR-2497, Added a new locator to the getPerson name method which will locate the required email address first before getting the person's name found in column 1. All tests pass as expected in a full regression test. (#1168) * PAR-2484 - Allow for Registered Organisations or Registered Charities… (#1160) * PAR-2484 - Allow for Registered Organisations or Registered Charities who have changed entity type since registration to be added as a legal entity - Adds a new option to allow users to add an entity who validation fails CH or CC lookups but saves on APIs requests * PAR-2490 - Correcting logic on adding new legal entity to an organisation * PAR-2487: Removing check to check if partnership is active to allow for legal entities to be edited before they are approved * PAR-2484: Temp Removing config readonly * PAR-2484: Updating validation on ne legal entities * Vault explore (#1163) * Update config.yml (#1161) Removes app checklist on db_sanitise as config import etc are not needed * Update config.yml * Update config.yml * Update config.yml * Update config.yml * Update config.yml * Update push.local.sh * Update config.yml * PAR-2491 & PAR-2492: Updating CF scripts to use new user creds * PAR-2491: More user switching * PAR-2491: Using same db import as stage fr the test env * Updating build scripts for new user variables * PAR-2493: Validation on partnership legal entity * PAR-2487: Removing checks on amendments if partnership is not confirmed as legal entities can now be added so this is redundant and blocking * PAR-2484: Removing lookup for companies house and charities commission fields * PAR-2487: Updating Legal entities to allow for varied numbers * Removing logic for saving issue * PAR-2495: Sanitising DB of Par Person data & PAR-2484: Fixing Legal Entity adding on confirmed partnerships. Will need to remove the line ../vendor/bin/drush spp && \ once this is in master so the daily job does the sanitisation * Swapping CircleCI IP ranges in * Fixing build tasks for CircleCI IP ranges * Updates for db backup * PAR-2484: Updating legal entities revoke permnission * PAR-2484: Resolving aggressive user checks * Updating open search * PAR-2498: Fixing Undefined variable $authorities in user profile update flow * PAR-2499: Fixing Legal entities permissions * Fixing missing display on Legal Entity updates --------- Co-authored-by: Tom <108397849+ThommCroft@users.noreply.github.com>
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.
No description provided.