Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Prod] Remove duplicative or unused tables from database #2185

Merged
merged 45 commits into from
Jun 5, 2024
Merged

[Prod] Remove duplicative or unused tables from database #2185

merged 45 commits into from
Jun 5, 2024

Conversation

thewatermethod
Copy link
Collaborator

@thewatermethod thewatermethod commented Jun 5, 2024 •
edited
Loading

Description of change

Now that we've reduced the fields in the RTR and confined others to the Activity Report only, we can reduce some of the tables designed to keep the two locations in sync.

How to test

Everything should function as before (except maybe a little bit faster).

Issue(s)

Checklists

Every PR

  • Meets issue criteria
  • JIRA ticket status updated
  • Code is meaningfully tested
  • Meets accessibility standards (WCAG 2.1 Levels A, AA)
  • API Documentation updated
  • Boundary diagram updated
  • Logical Data Model updated
  • Architectural Decision Records written for major infrastructure decisions
  • UI review complete

Before merge to main

  • OHS demo complete
  • Ready to create production PR

Production Deploy

  • Staging smoke test completed

After merge/deploy

  • Update JIRA ticket status

@thewatermethod
Nuke em boys
4dc08d7
@thewatermethod
Merge branch 'mb/TTAHUB-2660/activity-report-updates-for-reduced-obje…
ae7db1a 
…ctives' into mb/TTAHUB-2943/remove-models
@thewatermethod
remove hooks and some refs to deleted models
eb9010e
@thewatermethod
Remove unused models from report cache
eeafcce
@thewatermethod
Remove more refs
37a8074
@thewatermethod
Remove a bunch more refs
17f972a
@thewatermethod
Fix sessionReportPilotFile hook and add test
61ed320
@thewatermethod
Remove commented out test
b3e2088
@thewatermethod
Remove more refs
51da11e
@thewatermethod
Unused handlers
dc8556f
@thewatermethod
Remove some associations
cda0df3
@thewatermethod
Test updates and fixes
9abe3b1
@thewatermethod
Merge remote-tracking branch 'origin/main' into mb/TTAHUB-2943/remove…
9c4d6c0 
…-models
@thewatermethod
Update more tests
ad87b43
@thewatermethod
Delete empty file
5a3649c
@thewatermethod
Remove refs to dropped tables from "cleanupOrphanResources"
1c89e91
@thewatermethod
Fix API tests
0061b07
@thewatermethod
Fix topic rollup test
6303eb4
@thewatermethod
Fix assorted backend tests and overzealous deletions
8da4fdc
@thewatermethod
Patch objective reducer
b52bc6c
@thewatermethod
Fix e2e test
243e003
@thewatermethod
Updates for file uploader
ccffbe9
@thewatermethod
Remove findOrCreates from file services
825a281
@thewatermethod
fix test
e1768b1
@thewatermethod
Fix at least one of these tests
704e678
@thewatermethod
Merge branch 'mb/TTAHUB-2660/activity-report-updates-for-reduced-obje…
f53066b 
…ctives' into mb/TTAHUB-2943/remove-models
@thewatermethod
Hopefully resolves all errors
b3e984c
@thewatermethod
Remove log
daf04bd
@thewatermethod
Now this: this should work
541d104
@thewatermethod
Merge branch 'mb/TTAHUB-2660/activity-report-updates-for-reduced-obje…
c8d5a45 
…ctives' into mb/TTAHUB-2943/remove-models
@thewatermethod
Use migration helper and drop enums
51b2f21
@thewatermethod
Merge branch 'mb/TTAHUB-2660/activity-report-updates-for-reduced-obje…
7e92e96 
…ctives' into mb/TTAHUB-2943/remove-models
@thewatermethod
Update config.yml
33ac2de
@thewatermethod
Merge branch 'main' into mb/TTAHUB-2943/remove-models
a5c8c1f
@thewatermethod
Rename migration
69356a8
@thewatermethod
Rename migration
a04817d
@thewatermethod
Why not
587a765
@thewatermethod
Clear goal text when selected an initiative goal
81ad768
@thewatermethod
See if this makes this test less flaky
1475bbe
@thewatermethod
Create AROs if they do not yet exist
9f67c5f
@thewatermethod
Merge remote-tracking branch origin/main into mb/TTAHUB-2943/remove-m…
f939460 
…odels
@thewatermethod
Rename migration
3f25307
@thewatermethod
Merge remote-tracking branch origin/main into mb/TTAHUB-2943/remove-m…
0eb3c8b 
…odels
@thewatermethod
logical data-model
f7ab9a3
@thewatermethod
Merge pull request #2153 from HHS/mb/TTAHUB-2943/remove-models
041bf2d 
[TTAHUB-2943] Remove unneeded objective-related models
@thewatermethod thewatermethod marked this pull request as ready for review June 5, 2024 13:07
@Jones-QuarteyDana Jones-QuarteyDana merged commit 358ad1d into production Jun 5, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Jones-QuarteyDana Jones-QuarteyDana Jones-QuarteyDana approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@thewatermethod @Jones-QuarteyDana