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

release: v1.2.2 #90

Merged
merged 34 commits into from
Jul 9, 2024
Merged

release: v1.2.2 #90

merged 34 commits into from
Jul 9, 2024

Conversation

gzeoneth
Copy link
Member

@gzeoneth gzeoneth commented Jul 9, 2024

No description provided.

@gvladika
Scale retryable fee to fee token decimals denomination
a74b425
@gvladika
Approve the correct amount
fca3d6e
@gvladika
Merge branch 'add-e2e-tests' into non-18-decimals
645568e
@gvladika
Provide retryable fee for creating child chain contract from creator …
bc685e9 
…to retryable sender
@gvladika
Remove debug logs
c3de1de
@gvladika
Move back struct
886215d
@gvladika
Move to internal function
745bfa5
@gvladika
Call _getFeeToken only once, lower the contract size
d9142a0
@gvladika
Calculate fee amount denominated in fee token decimals
f22e38e
@gvladika
Fix default gateway deposit test
7870283
@gvladika
Fix custom gateway deposit test
9ab28f6
@gvladika
Update slither db
44bad07
@gvladika
Refactor _getScaledAmount to decreaase bytecode size
6eb6ac4
@gvladika
Signature change in retryable sender
299c9a0
@gvladika
Use current token bridge branch in testnode
5b3fd89
@gvladika
Use temporarily custom branch for CI
e928fef
@gvladika
Add CI job that runs over 6-decimals fee token chain
183929b
@gvladika
Fix param name
9cf47a3
@gvladika
Use custom testnode branch for decimals test
37ee02a
@gvladika
Add unit tests to test 6 decimals fee token
d85762a
@gvladika
Add fuzz test for custom number of decimals in fee token
07f5fe5
@gvladika
Merge branch 'add-e2e-tests' into non-18-decimals
f5d4459
@gvladika
Merge branch 'add-e2e-tests' into non-18-decimals
eb12508
@gvladika
Update slither db
2dab492
@gvladika
Merge branch 'main' into non-18-decimals
8b5c834
@gvladika
Bump nitro-contract for test
5c22f30
@gvladika
Use temporary for test until nitrp-contract official relese is bumped
657972f
@gvladika
Update branch
61224dc
@gvladika
Bump node in CI
01aa7fd
@gvladika
Node18 expects 127.0.0.1 instead of localhost
bdd49cd
@cla-bot cla-bot bot added the cla-signed label Jul 9, 2024
@gzeoneth gzeoneth requested a review from gvladika July 9, 2024 14:24
gvladika
gvladika approved these changes Jul 9, 2024
View reviewed changes
Copy link
Contributor

@gvladika gvladika left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@gzeoneth gzeoneth merged commit 9b5d71a into main Jul 9, 2024
10 checks passed
@gzeoneth gzeoneth deleted the develop branch July 9, 2024 14:43
@gzeoneth gzeoneth restored the develop branch July 9, 2024 14:43
@queenmoon-bot
Copy link

Add

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gvladika gvladika gvladika approved these changes

Assignees
No one assigned
Labels
cla-signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gzeoneth @queenmoon-bot @gvladika