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

Make N3 Gas Claims look like Gas Claims on the Activity Page #2470

Merged
merged 4 commits into from
Mar 23, 2023

Conversation

jseagrave21
Copy link
Contributor

@jseagrave21 jseagrave21 commented Mar 12, 2023

What current issue(s) from Trello/Github does this address?
#2424

What problem does this PR solve?
Makes N3 Gas Claims look like Gas Claims on the Activity page

How did you solve this problem?
Updated transactionHistoryActions to look for a Gas Claim, then it updates the metadata so the current code sees it as a Gas Claim. Then I added N3ClaimAbstract so it looks right on the Activity Page.

How did you make sure your solution works?
yarn dev testing and verification with the unit test.

Are there any special changes in the code that we should be aware of?
No.

Is there anything else we should know?
This change does not fully update the tx metadata. I only updated enough to make the tx look like it should on the Activity page.

  • Unit tests written?

@jseagrave21
Copy link
Contributor Author

@comountainclimber I'm not sure how to get the checks to run remotely. Everything passes locally.

@comountainclimber
Copy link
Member

@jseagrave21 this is awesome thanks for contributing!! I has some minor feedback, basically I think we can get away with making the N3ClaimAbstract.jsx component much less verbose.

No idea whats going on with CI but hopefully when you push up another commit it will trigger the test suite.

NICE WORK 🤩

@jseagrave21
Copy link
Contributor Author

jseagrave21 commented Mar 16, 2023

@comountainclimber thank you! I agree. I followed the example from the N3VoteAbstract and trimmed away as much as I think makes sense. Let me know what you think.
Gas Claim

@comountainclimber comountainclimber merged commit 8f702db into CityOfZion:dev Mar 23, 2023
@jseagrave21 jseagrave21 deleted the n3_gas_claim_activity branch March 23, 2023 19:28
@ixje
Copy link
Member

ixje commented Mar 24, 2023

🎉 nice!

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

Successfully merging this pull request may close these issues.

3 participants