-
Notifications
You must be signed in to change notification settings - Fork 5.4k
feat: implement swap quote received event #4165
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
Merged
Merged
Changes from all commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
11fc709
init commit
0864ee4
add amplitude ts sdk to package.json
f9f013d
add more comments and documentation
561aadf
respond to vm comments
682d713
respond to cmcewen comments
4d3f629
fix: remove unused constants
6e5119d
init commit
8f185b3
adapt to web
02b927c
add optional event properties to trace
f5ad7f1
correct telemetry to analytics
4fe3aa2
init commit
2c40799
change telemetry to analytics in doc
84e9683
init commit
42fddee
fix: respond to cmcewen comments + initialize analytics in app.tsx + …
40b148b
Merge branch 'amplitude_sdk_initial' into analytics_trace_framework
3f01dd7
Merge branch 'analytics_trace_framework' into WEB-214-implement-the-m…
5ee155d
add element name constant
caa99fc
merge
5fad981
Merge branch 'main' into WEB-214-implement-the-main-submit-swap-event
f40e223
Merge branch 'main' into WEB-214-implement-the-main-submit-swap-event
9232948
init commit
8072ae2
Merge branch 'main' into WEB-214-implement-the-main-submit-swap-event
8db5a4c
correct price_impact calculation
3cc77be
resolve vm comments
23ab7af
fixes in response to comments
d314f77
respond to vm
ec51b9a
use ALL significant digits for token amounts
519b1cb
init commit
3ed403c
Merge branch 'main' into WEB-214-implement-the-main-submit-swap-event
39f6073
merge main
d36824d
Merge branch 'WEB-214-implement-the-main-submit-swap-event' into swap…
0e6c7df
logged all properties
dc6fa77
create helper function getPriceImpactPercentageNumber
34a72c0
4 decimal points for percentages
d4f7352
Merge branch 'WEB-214-implement-the-main-submit-swap-event' into swap…
cf83cea
price percentage fn
a4ddbbf
percentage to basis points units
4c18c39
only log event on FIRST price fetch
8b222ea
respond to cmcewen comments
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| import { Currency, CurrencyAmount, Percent, Token } from '@uniswap/sdk-core' | ||
|
|
||
| export const getDurationTillTimestampSinceEpoch = (futureTimestampSinceEpoch?: number): number | undefined => { | ||
| if (!futureTimestampSinceEpoch) return undefined | ||
| return futureTimestampSinceEpoch - new Date().getTime() / 1000 | ||
| } | ||
|
|
||
| export const getNumberFormattedToDecimalPlace = ( | ||
| intialNumberObject: Percent | CurrencyAmount<Token | Currency>, | ||
| decimalPlace: number | ||
| ): number => parseFloat(intialNumberObject.toFixed(decimalPlace)) | ||
|
|
||
| export const formatPercentInBasisPointsNumber = (percent: Percent): number => parseFloat(percent.toFixed(2)) * 100 |
This file contains hidden or 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
This file contains hidden or 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
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.
Uh oh!
There was an error while loading. Please reload this page.