-
Notifications
You must be signed in to change notification settings - Fork 596
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 ledger #5233
Merged
skylarbarrera
merged 9 commits into
@benisgold/points-tab-ui
from
@skylar/points-4-ledger
Dec 11, 2023
Merged
add ledger #5233
skylarbarrera
merged 9 commits into
@benisgold/points-tab-ui
from
@skylar/points-4-ledger
Dec 11, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
walmat
reviewed
Dec 11, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some logs cleanup, can't test this due to not having a Ledger, but code looks good to me.
walmat
approved these changes
Dec 11, 2023
skylarbarrera
added a commit
that referenced
this pull request
Dec 11, 2023
* points tab ui * i18n * fix * fix android * android fix * Points: hook up points tab ui to metadata api (#5176) * points hook * some error state + pull to refresh * fix copy haptic + pull to refresh * fix * Points: introductory flow (#5201) * refactor * claim logic * mutation * claim content * referral screen * referral ui * temp * fix android input and tabs * fix animation + back button android * layout fixes * final fixes + i18n * Points: signing up (#5210) * Add SF Mono, SF Pro Rounded Black * Add AnimatePresence * Update green/red colors * Add ConsoleSheet * Hook up iOS fonts * Hook up to PointsScreen * Style fixes * hook up some logic to claim signing * no retroactive points ui * fix points content layout * fix * i18n * fix error logging --------- Co-authored-by: Christian Baroni <7061887+christianbaroni@users.noreply.github.com> * remove obsolete useFocusEffect * ImgixImage * simulationClient -> metadataPOSTClient --------- Co-authored-by: Christian Baroni <7061887+christianbaroni@users.noreply.github.com> --------- Co-authored-by: Christian Baroni <7061887+christianbaroni@users.noreply.github.com> * merge fix * Points fixes (#5219) * points hook * some error state + pull to refresh * fix copy haptic + pull to refresh * refactor * claim logic * fix * mutation * Add SF Mono, SF Pro Rounded Black * Add AnimatePresence * Update green/red colors * Add ConsoleSheet * Hook up iOS fonts * Hook up to PointsScreen * Style fixes * claim content * referral screen * referral ui * temp * fix android input and tabs * fix animation + back button android * layout fixes * final fixes + i18n * hook up some logic to claim signing * no retroactive points ui * fix points content layout * fix * animiation fix * referral code ui * fix invalid referral error * remove leaderboard row nav to profile * remove obsolete card + add rank card * black font weight for total points * refetch if past next distribution date * change rank icon to support higher ranks * leaderboard up to rank 100 * fix black font android * tweak sf mono font * dont render ens nft avatars in leaderboard * ActionButton * points icon animation * fix points gradient * format total # points users * restrict read only wallets * copy * referral code query update * fixing referral validation * fix merge error * fix referral code input * change metadata api fetch method from post to get * routes * i18n * console sheet nav --------- Co-authored-by: Christian Baroni <7061887+christianbaroni@users.noreply.github.com> * add contentful points tweet intent schema and queries (#5228) * i18n * real referral url * rm TAB_BAR_HEIGHT * Add referral share step (#5229) * add contentful points tweet intent schema and queries * progress on points share screen * finish flow * clean up code and break it into smaller files * refactor progress * final changes * finished up * lint * revert graphql endpoint test * share bonus points --------- Co-authored-by: Ben Goldberg <bengoldberg@rainbow.me> * fix referral skeleton ui * fix android NeonButton * cleanup * types * add 3 dots back to calculating points * fix share link * fix android shadow clipping * fix continue button not showing * [APP-972]: Fix back button being under the bottom tab bar (#5231) * fix continue button not showing * fix back button being under the bottom tabbar * 1000 -> 100 bonus pts * add a hardcoded version of tweet intent to build copy (#5235) * fix placeholder on android (#5234) * add proper twitter intents * change intent slightly * fix: rainbow text spacing and font issues * fix fonts * Bug bash fixes (#5237) * additional onboard error handling * fix skip * copy * bump referral keyboard delay * fix double referral link when sharing * fix next drop card date conversion * tweak refetch interval slightly * twitter intent tweak * fix console animation + hide mm row if 0 * console button animation delay * tweak twitter intent * bonus points -> welcome bonus || true believer * block hardware wallet * leaderboard row onpress + refactor * next drop card tweak * deeplink * referral bug fix * fix android referral code text color * attempt to fix ledger sign on console sheet * analytics events * fix android points mask * add ledger (#5233) * add ledger * logs * logs * logs * analytics events * tweaks * logs * moar logs --------- Co-authored-by: Ben Goldberg <bengoldberg@rainbow.me> * remove url encoded (#5240) * added analytics events * next drop rounding fix * rm console logs --------- Co-authored-by: Christian Baroni <7061887+christianbaroni@users.noreply.github.com> Co-authored-by: Matthew Wall <matthew.wallt@gmail.com> Co-authored-by: Derek <dereknelsony0@gmail.com> Co-authored-by: Skylar Barrera <skylar.barrera@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes APP-970
What changed (plus any additional context for devs)
Screen recordings / screenshots
https://cloud.skylarbarrera.com/RPReplay_Final1702316398.MP4
What to test