-
Notifications
You must be signed in to change notification settings - Fork 16
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
Feature/bugfix/amount type #18
Closed
Closed
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
I've started a standard VueJS project including features to code using the AirBnB JS eslint config, unit tests and E2E tests, as well as key features for Vue such as routing and App State using vue-router and vuex. We can leverage the project starting from this base.
We have migrated the project to support TypeScript according Federation Team recommendation.
I've added vuetify library to the project.
I've added all the compnents avaible on the branch home, now using type script.
I've added all the design elements to complete the select bitcoin wallet step.
I've added all the screen to get all the information needed to make the peg-in.
We've added the default configuration for png files on typescript
I've added the content for the transaction status view and added somo of the screens after completing pegin on exchange for ledger wallet.
I've added the tracking id pop-up after succesfully send the peg in transaction.
We've added vuex config for typescript project
I've created the trezor service in order to extract the account information fron the device
I've change the styles and removes Drawer element to match the new design of the home view.
I've added the Service abstraction for tx builder and signer on every wallet.
I've implementent all the views to make trezor pegin flow with the new designs.
I've integrated Trezor service getAccountUtxos funtion to retrieve utxos information from trezor wallet.
We've added the listener for the trezor events.
We've fixed some of the view of the pegin flow on trezor.
I've used the get pegin configuration and get balances end points from de API, and include them on the trezor flow.
I've made some style improvements on the pegin form.
We've added the tx-fee and pegin-tx endpoinst on Api
fixing errors on build
go back button is also disabled while confirming on device
Co-authored-by: josedahlquist <[email protected]>
Updating status view observations
Change spinners behavior and show RSK federation address
fix text line max lenght allowed on tx summary
Fix step selection on send bitcoin form
I've updated the percentage calculation based on the peginConfiguration confirmation values for the required network
Updating Tx status plot
computation is safer and using big js
* no letters allowed * no amount over 99_999_999 * limit of 8 decimals note: it does not prevent actual input, it just disables Send button
Send button disables when amount is filled last and has incorrect values
initial balance was taken from native segwit address before user selected an address type
Form validation and user input
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.
No description provided.