Skip to content
This repository has been archived by the owner on Jun 19, 2024. It is now read-only.

Update views tx_vote_proposal and tx_transfer #185

Merged
merged 4 commits into from
Apr 10, 2024
Merged

Conversation

opsecx
Copy link
Contributor

@opsecx opsecx commented Apr 5, 2024

The views tx_vote_proposal and tx_transfer took in all transactions regardless of status (failed/succeeded). Updated views to only use successful transactions, as in former version tables. Note: An alternate approach might be to make the views richer and include fail/success status in tables. For now this is a quick fix.

(I assume this may be an issue with the other views as well, but not familiar enough with the data to fix atm. I have validated the data from this particular fix against former databases and seen that it's better corresponding)

@opsecx
Copy link
Contributor Author

opsecx commented Apr 5, 2024

I may have made things difficult by not removing a former commit, not sure if that's problematic for revision history, not sure how to fix that. Lmk if needs changing

@rllola
Copy link
Contributor

rllola commented Apr 9, 2024

There is no conflict so we should be good.

@opsecx
Copy link
Contributor Author

opsecx commented Apr 9, 2024

I see there's a build failing in checks, but not familiar enough with process to know if significant.

@rllola
Copy link
Contributor

rllola commented Apr 10, 2024

Don't worry it is earthly. It is part of our devops process and it is not required.

@rllola rllola merged commit 2f445d5 into Zondax:main Apr 10, 2024
5 of 6 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants