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

Add service fee claim page #733

Merged
merged 36 commits into from
Jun 12, 2024
Merged

Add service fee claim page #733

merged 36 commits into from
Jun 12, 2024

Conversation

pivilartisant
Copy link
Contributor

No description provided.

@pivilartisant pivilartisant changed the base branch from main to integration June 10, 2024 16:03
@pivilartisant pivilartisant marked this pull request as draft June 10, 2024 16:03
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-733.d1u92gh798a6e1.amplifyapp.com

src/utils/utils.ts Outdated Show resolved Hide resolved
src/utils/utils.ts Outdated Show resolved Hide resolved
@pivilartisant pivilartisant marked this pull request as ready for review June 11, 2024 09:47
@pivilartisant pivilartisant requested a review from Thykof June 11, 2024 12:05
@pivilartisant pivilartisant force-pushed the add-service-fee-claim-page branch 2 times, most recently from 21e2909 to 0af22b2 Compare June 11, 2024 13:41
@pivilartisant pivilartisant force-pushed the add-service-fee-claim-page branch from 13f6bdc to 4246af1 Compare June 11, 2024 15:09
@pivilartisant pivilartisant force-pushed the add-service-fee-claim-page branch from f7f8c0f to 13c2648 Compare June 11, 2024 17:01
src/utils/utils.ts Outdated Show resolved Hide resolved
@pivilartisant pivilartisant requested a review from Thykof June 12, 2024 10:00
src/pages/ClaimPage/SuccessClaim.tsx Outdated Show resolved Hide resolved
src/pages/ClaimPage/InitClaim/InitClaim.tsx Outdated Show resolved Hide resolved
src/custom/bridge/useHandleBurnRedeem.ts Outdated Show resolved Hide resolved
src/pages/HistoryPage/Amount.tsx Outdated Show resolved Hide resolved
src/store/operationStore.ts Outdated Show resolved Hide resolved
src/utils/utils.ts Outdated Show resolved Hide resolved
src/pages/ClaimPage/InitClaim/InitClaim.tsx Outdated Show resolved Hide resolved
src/pages/ClaimPage/SuccessClaim.tsx Outdated Show resolved Hide resolved
src/utils/utils.ts Outdated Show resolved Hide resolved
src/utils/utils.ts Outdated Show resolved Hide resolved
@pivilartisant pivilartisant force-pushed the add-service-fee-claim-page branch from 550f92a to 210e62e Compare June 12, 2024 14:14
@pivilartisant pivilartisant force-pushed the add-service-fee-claim-page branch from 210e62e to 72fdf7c Compare June 12, 2024 14:19
@Thykof Thykof merged commit 91f2d81 into integration Jun 12, 2024
2 of 3 checks passed
@Thykof Thykof deleted the add-service-fee-claim-page branch June 12, 2024 14:20
peterjah pushed a commit that referenced this pull request Jul 16, 2024
* add service fees on claim page

* add success amount

* update usehandleBurnRedeem hook

* clean utils

* move logic to initClaim.tsx

* clean

* rename amount to receive

* add outputAmount check in succesClaim

* refacto getAmountToReceive and edit tests

* refactor getAmountToreceive

* remove undefined from success claim props

* add comments

* refactor input and output

* add outputAmount to success page

* clean bridgeRedeemLayout

* fix changeAmount fn

* Revert "fix changeAmount fn"

This reverts commit 13c2648.

* clean changeAmount fn

* set inputs undefined

* exp => inputAmount to bigint

* update ui kit with npm link, PR #459

* fix retrievePercent

* fix Amount

* fix tests

* rename variable

* rename var + fix component

* fix claim page wwhen failing to parse 1,000 to bigint

* remove comment

* prevent crashing from parsing a string with , seperator

* seperate send from received

* refactor getAmountReceived()

* refactor getAmountReceived() p2

* clean claim page

* rename amount in BridgeRedeemLayout.tsx

* compare struct equal and fix test

* rename _outputAmount

---------

Co-authored-by: Nathan Seva <[email protected]>
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.

2 participants