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.

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

Remove committee transaction views from DB #1232

Open
exalate-issue-sync bot opened this issue Dec 2, 2024 · 0 comments
Open

Remove committee transaction views from DB #1232

exalate-issue-sync bot opened this issue Dec 2, 2024 · 0 comments
Assignees

Comments

@exalate-issue-sync
Copy link

exalate-issue-sync bot commented Dec 2, 2024

We’ve identified that there is no longer a need to use DB views as a layer above our transaction table. It’s adding problems to our deployments and general complexity.

Remove the creation of committee transaction views and move sql logic back into the base transaction manager

QA Notes

Verify unit test passing, and that there are no new gaps introduced.

original implementation of views: [https://github.com//pull/769/files#diff-d56ea0f167f0f4f4406e7166d1d9e3646ced77d96105c2589f0f40af87024457|https://github.com//pull/769/files#diff-d56ea0f167f0f4f4406e7166d1d9e3646ced77d96105c2589f0f40af87024457|smart-link]

DEV Notes

null

Design

null

See full ticket and images here: FECFILE-1869

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants