Production release #766
Merged
Production release #766
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
Merge Master into Dev.
…sactions bugfix - avoid using null contract
…connection-after-refresh-if-user-refused-to-connect-once #715 | fix Metamask popup problem
✅ Deploy Preview for testnet-teloscan ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for teloscan ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for dev-mainnet-teloscan ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for teloscan-stage ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
#737 | workaround for the NaN age + fix false values not showing
…w-contract-function-call #747 | decode trx input to show contract function call
…d-message-display-bug #739 | TransactionAction cases: Function call, TLOS transfer, Contract Creation
…eview-image-on-click-1 #740 | Popup Image viewer for NFTs
rozzaswap
approved these changes
Jun 11, 2024
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 #755
Fixes #715
Fixes #737
Fixes #760
Fixes #747
Fixes #740
Fixes #739
Fixed Metamask Popup Issue
In the previous version, after a not finished attempt to login using Metamask, the user experienced a constant and annoying behavior of the Metamask popup appearing each time the page gets refreshed.
This PR fixes this problem.
Prevented Null Contract Usage
The transaction page was failing to show Internal Transactions for the following case:
Example Transaction
Invalid Age on Transactions Table for an Address
This PR adds a workaround for the NaN age issue but references telosnetwork/teloscan-indexer#234 since it can appear elsewhere.
Bool Read Function Not Returning a Value
This PR also adds a fix for the bug in the boolean queries not showing the false value. This is fixed, and now both true and false values are displayed.
Decode Transaction Input to Show Contract Function Call
This PR includes the solution for two of the three views that the new
TransactionInputViewercomponent should offer. The Default view requires a refinement of the algorithms, so it was separated into this issue #762 and will be scheduled apart.The two views implemented are:
NFT Viewer Popup
This PR adds a new component to preview the image in a popup dialog. This dialog can be resized and also includes a button to download the image.
Transaction action improvements
The TransactionAction component was re-implemented into several cases to display differently according to each case.
Testing details: #742