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

GraphPayments changes post audit #1078

Merged
merged 7 commits into from
Feb 10, 2025
Merged

GraphPayments changes post audit #1078

merged 7 commits into from
Feb 10, 2025

Conversation

tmigone
Copy link
Contributor

@tmigone tmigone commented Dec 11, 2024

No description provided.

@tmigone tmigone changed the title feat: extract cancel thaw escrow to its own function GraphPayments changes post audit Dec 11, 2024
@openzeppelin-code OpenZeppelin Code
Copy link

openzeppelin-code bot commented Dec 11, 2024 •
edited
Loading

GraphPayments changes post audit

Generated at commit: 20396ba44cfc624443d72e436d02a9ebe8101822

🚨 Report Summary

Severity Level Results
Contracts Critical
High
Medium
Low
Note
Total		 2
5
0
15
39
61
Dependencies Critical
High
Medium
Low
Note
Total		 0
0
0
0
0
0

For more details view the full report in OpenZeppelin Code Inspector

@tmigone tmigone marked this pull request as draft December 17, 2024 19:52
@tmigone tmigone force-pushed the tmigone/post-trust-changes-payments branch from d3318eb to f056410 Compare December 18, 2024 16:59
@tmigone tmigone changed the base branch from tmigone/trust-fixes-payments to horizon December 18, 2024 17:00
@tmigone tmigone marked this pull request as ready for review January 27, 2025 18:13
@tmigone tmigone requested a review from Maikol January 27, 2025 18:13
@tmigone tmigone force-pushed the tmigone/post-trust-changes-payments branch 2 times, most recently from 9a1a773 to 443b1ef Compare February 7, 2025 20:31
@tmigone
feat: extract cancel thaw escrow to its own function
6d1afc0 
Signed-off-by: Tomás Migone <tomas@edgeandnode.com>
@tmigone
feat: add collector address to the RAV
74645a2 
Signed-off-by: Tomás Migone <tomas@edgeandnode.com>
@tmigone
test: improve coverage for GraphPayments
744bc85 
Signed-off-by: Tomás Migone <tomas@edgeandnode.com>
@tmigone
chore: some minor improvements to GraphPayments and tests
eaec959 
Signed-off-by: Tomás Migone <tomas@edgeandnode.com>
@tmigone
fix: couple minor improvments to PaymentsEscrow and some new tests
9b79fdb 
Signed-off-by: Tomás Migone <tomas@edgeandnode.com>
@tmigone
chore: revert adding collector address to RAV
90978a9 
Signed-off-by: Tomás Migone <tomas@edgeandnode.com>
@tmigone
chore: revert adding collector address to RAV in SubgraphService
20396ba 
Signed-off-by: Tomás Migone <tomas@edgeandnode.com>
@tmigone tmigone force-pushed the tmigone/post-trust-changes-payments branch from 443b1ef to 20396ba Compare February 10, 2025 16:02
@tmigone tmigone merged commit 51976ec into horizon Feb 10, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Maikol Maikol Maikol approved these changes

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

Successfully merging this pull request may close these issues.
2 participants
@tmigone @Maikol