-
Notifications
You must be signed in to change notification settings - Fork 0
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
GAP-2533: Retrospectively fix BST ads in prod #276
Merged
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
dylanwrightCO
approved these changes
Apr 2, 2024
dominicwest
added a commit
that referenced
this pull request
Apr 2, 2024
* GAP-2533: Retrospectively fix BST ads in prod * GAP-2533: Including release time * GAP-2533: Overriding UTC opening/closing dates as Europe/London time * GAP-2533: Update scheduled closing times too * GAP-2533: Fix syntax * GAP-2533: Hard code datetimes for BST
arul-fourseals
added a commit
that referenced
this pull request
Jan 8, 2025
* Release/3.2 (#28) * Adding constraint on gap_admin table * Adding unique constraint on grant_applicant_organisation_profile table * Removing unique constraints on tables with foreign key constraints * Release/3.3 (#39) Release 3.3 * Release/3.6 (#47) * TMI2-253: Add pre-commit check and update .gitignore to include application-local.properties and .env.local * Fixing unique constraint on indexes (#33) * Fixing unique constraint on indexes * GAP-2070 delete applicant from apply DB (#15) Hard deletes an applicant after an admin deletes them * GAP-2070 grant_beneficiary fk constraint & set null delete cascade (#34) * GAP-2070 fixing not null constraint on created_by * GAP-2148: grant advert not publishing on selected date (#32) * GAP-2148 - fixing view to use UTC * GAP-2148 - fixing scheduled job & setting opening/closing dates to use UTC not BST (#40) - Removing code that formats the excel sheet as this was causing an unacceptable performance problem * GAP-2230: Removing transactional from inserting exports, as this results in the lambda being invoked before exports exist in the DB (#46) * Creates an endpoint which removes applications attached to an advert (#42) (#49) --------- Co-authored-by: paul-lawlor-tco <[email protected]> Co-authored-by: Iain Cooper <[email protected]> Co-authored-by: iaincooper-tco <[email protected]> Co-authored-by: john-tco <[email protected]> Co-authored-by: rachelswart <[email protected]> Co-authored-by: ConorFayleTCO <[email protected]> Co-authored-by: GavCookCO <[email protected]> Co-authored-by: paul-lawlor-tco <[email protected]> * Create DUMMY * Tmi2 501/revert tmi2 496 changes (#129) (#130) * revert tmi2-496 changes * format * Release/6.1 (#136) * Tmi2 501/revert tmi2 496 changes (#129) * revert tmi2-496 changes * format * Adding tables for completion_statistics.sql (#131) * Adding tables for completion_statistics.sql * Formatting... * Tn 154 calculations tables (#132) * Adding tables for completion_statistics.sql * Formatting... * Adding sequence for calculations table id * Tn 154 calculations tables (#133) * Adding tables for completion_statistics.sql * Formatting... * Adding sequence for calculations table id * Fixing incorrect format for migration script name. Combining 2 into 1 * Fixing issue with db migration script (#134) * Adding tables for completion_statistics.sql * Formatting... * Adding sequence for calculations table id * Fixing incorrect format for migration script name. Combining 2 into 1 * Fixing errors in db migration script --------- Co-authored-by: a-lor-cab <[email protected]> * Fix promoteToProd.yml (#179) * Adds query to find application with no ownership check to allow lambdas to call (#212) * Fix unpublish application form from lambda (#268) (#269) * Updating release 10.0 with develop changes (#274) * Fix unpublish application form from lambda (#268) * GAP-2531: Cascade delete grant export tables createdby column (#270) * GAP-2533: Fix advert scheduler view for BST (#273) * use LocalDateTime for grant opening and closing times (#272) * use LocalDateTime for grant opening and closing times * fix test * fix more tests * Tmi2 740 - Handling new locations for MQ saving to grant_beneficiary table (#271) * Adding two new columns to grant_beneficiary for new locations --------- Co-authored-by: jgunnCO <[email protected]> Co-authored-by: Connor Macqueen <[email protected]> * GAP-2533: Retrospectively fix BST ads in prod (#276) (#278) * GAP-2533: Retrospectively fix BST ads in prod * GAP-2533: Including release time * GAP-2533: Overriding UTC opening/closing dates as Europe/London time * GAP-2533: Update scheduled closing times too * GAP-2533: Fix syntax * GAP-2533: Hard code datetimes for BST * Improves logs around updating and deleting open search (#280) * Improving logging (#283) * Improving logging II * amend schemeeditorcontroller * Release/12.0 (#308) * amend schemeeditorcontroller (#302) * url validation updated to accept # symbol * existing unknown chane removed (#307) --------- Co-authored-by: jgunnCO <[email protected]> * convert admin email to lowercase --------- Co-authored-by: dominicwest <[email protected]> Co-authored-by: dylanwrightCO <[email protected]> Co-authored-by: paul-lawlor-tco <[email protected]> Co-authored-by: Iain Cooper <[email protected]> Co-authored-by: iaincooper-tco <[email protected]> Co-authored-by: john-tco <[email protected]> Co-authored-by: rachelswart <[email protected]> Co-authored-by: ConorFayleTCO <[email protected]> Co-authored-by: GavCookCO <[email protected]> Co-authored-by: Gavin Cook <[email protected]> Co-authored-by: JamieGunnCO <[email protected]> Co-authored-by: a-lor-cab <[email protected]> Co-authored-by: Connor Macqueen <[email protected]> Co-authored-by: dylanwrightCO <[email protected]> Co-authored-by: jgunnCO <[email protected]>
arul-fourseals
added a commit
that referenced
this pull request
Jan 9, 2025
* amend schemeeditorcontroller (#302) * url validation updated to accept # symbol (#305) * Fix/gap lss 431 (#306) * url validation updated to accept # symbol * existing unknown chane removed * Lss 701 ownership transfer case sensitive (#309) * Release/3.2 (#28) * Adding constraint on gap_admin table * Adding unique constraint on grant_applicant_organisation_profile table * Removing unique constraints on tables with foreign key constraints * Release/3.3 (#39) Release 3.3 * Release/3.6 (#47) * TMI2-253: Add pre-commit check and update .gitignore to include application-local.properties and .env.local * Fixing unique constraint on indexes (#33) * Fixing unique constraint on indexes * GAP-2070 delete applicant from apply DB (#15) Hard deletes an applicant after an admin deletes them * GAP-2070 grant_beneficiary fk constraint & set null delete cascade (#34) * GAP-2070 fixing not null constraint on created_by * GAP-2148: grant advert not publishing on selected date (#32) * GAP-2148 - fixing view to use UTC * GAP-2148 - fixing scheduled job & setting opening/closing dates to use UTC not BST (#40) - Removing code that formats the excel sheet as this was causing an unacceptable performance problem * GAP-2230: Removing transactional from inserting exports, as this results in the lambda being invoked before exports exist in the DB (#46) * Creates an endpoint which removes applications attached to an advert (#42) (#49) --------- Co-authored-by: paul-lawlor-tco <[email protected]> Co-authored-by: Iain Cooper <[email protected]> Co-authored-by: iaincooper-tco <[email protected]> Co-authored-by: john-tco <[email protected]> Co-authored-by: rachelswart <[email protected]> Co-authored-by: ConorFayleTCO <[email protected]> Co-authored-by: GavCookCO <[email protected]> Co-authored-by: paul-lawlor-tco <[email protected]> * Create DUMMY * Tmi2 501/revert tmi2 496 changes (#129) (#130) * revert tmi2-496 changes * format * Release/6.1 (#136) * Tmi2 501/revert tmi2 496 changes (#129) * revert tmi2-496 changes * format * Adding tables for completion_statistics.sql (#131) * Adding tables for completion_statistics.sql * Formatting... * Tn 154 calculations tables (#132) * Adding tables for completion_statistics.sql * Formatting... * Adding sequence for calculations table id * Tn 154 calculations tables (#133) * Adding tables for completion_statistics.sql * Formatting... * Adding sequence for calculations table id * Fixing incorrect format for migration script name. Combining 2 into 1 * Fixing issue with db migration script (#134) * Adding tables for completion_statistics.sql * Formatting... * Adding sequence for calculations table id * Fixing incorrect format for migration script name. Combining 2 into 1 * Fixing errors in db migration script --------- Co-authored-by: a-lor-cab <[email protected]> * Fix promoteToProd.yml (#179) * Adds query to find application with no ownership check to allow lambdas to call (#212) * Fix unpublish application form from lambda (#268) (#269) * Updating release 10.0 with develop changes (#274) * Fix unpublish application form from lambda (#268) * GAP-2531: Cascade delete grant export tables createdby column (#270) * GAP-2533: Fix advert scheduler view for BST (#273) * use LocalDateTime for grant opening and closing times (#272) * use LocalDateTime for grant opening and closing times * fix test * fix more tests * Tmi2 740 - Handling new locations for MQ saving to grant_beneficiary table (#271) * Adding two new columns to grant_beneficiary for new locations --------- Co-authored-by: jgunnCO <[email protected]> Co-authored-by: Connor Macqueen <[email protected]> * GAP-2533: Retrospectively fix BST ads in prod (#276) (#278) * GAP-2533: Retrospectively fix BST ads in prod * GAP-2533: Including release time * GAP-2533: Overriding UTC opening/closing dates as Europe/London time * GAP-2533: Update scheduled closing times too * GAP-2533: Fix syntax * GAP-2533: Hard code datetimes for BST * Improves logs around updating and deleting open search (#280) * Improving logging (#283) * Improving logging II * amend schemeeditorcontroller * Release/12.0 (#308) * amend schemeeditorcontroller (#302) * url validation updated to accept # symbol * existing unknown chane removed (#307) --------- Co-authored-by: jgunnCO <[email protected]> * convert admin email to lowercase --------- Co-authored-by: dominicwest <[email protected]> Co-authored-by: dylanwrightCO <[email protected]> Co-authored-by: paul-lawlor-tco <[email protected]> Co-authored-by: Iain Cooper <[email protected]> Co-authored-by: iaincooper-tco <[email protected]> Co-authored-by: john-tco <[email protected]> Co-authored-by: rachelswart <[email protected]> Co-authored-by: ConorFayleTCO <[email protected]> Co-authored-by: GavCookCO <[email protected]> Co-authored-by: Gavin Cook <[email protected]> Co-authored-by: JamieGunnCO <[email protected]> Co-authored-by: a-lor-cab <[email protected]> Co-authored-by: Connor Macqueen <[email protected]> Co-authored-by: dylanwrightCO <[email protected]> Co-authored-by: jgunnCO <[email protected]> --------- Co-authored-by: jgunnCO <[email protected]> Co-authored-by: dominicwest <[email protected]> Co-authored-by: dylanwrightCO <[email protected]> Co-authored-by: paul-lawlor-tco <[email protected]> Co-authored-by: Iain Cooper <[email protected]> Co-authored-by: iaincooper-tco <[email protected]> Co-authored-by: john-tco <[email protected]> Co-authored-by: rachelswart <[email protected]> Co-authored-by: ConorFayleTCO <[email protected]> Co-authored-by: GavCookCO <[email protected]> Co-authored-by: Gavin Cook <[email protected]> Co-authored-by: JamieGunnCO <[email protected]> Co-authored-by: a-lor-cab <[email protected]> Co-authored-by: Connor Macqueen <[email protected]> Co-authored-by: dylanwrightCO <[email protected]>
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.