forked from blockscout/blockscout
-
Notifications
You must be signed in to change notification settings - Fork 20
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
W35 upstream merge #369
Merged
Merged
W35 upstream merge #369
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
Search results page
Fix routes: add NETWORK_PATH variable to current_path
…ns-step-to-docker-start-script Add DB migrations step into Docker start script
Change order of transations in block's view
…en-transfers-query-performance-improvement Uncataloged token transfers query performance improvement
…nces-fix Indexer performance update: refactor coin balance daily fetcher
Add transaction's index
Update txn page tooltips
Add arbitrum support
…dress_token_balance view. (#359)
…med_tokens Use token_symbol helper when no symbol exists for a given token in ad…
Indexer performance update: Eliminate multiple updates of the same token while parsing mint/burn token transfers batch
…cify_metadata_json Search for metadata.json file in sourcify response.
…n_balance_fetch Access token type from token object.
This reverts commit d8d3507.
…compile_tests Upstream/bugfix/mix compile tests
…p-genesis-block Indexer performance update: skip genesis block in requesting of trace_block API endpoint
…tails Rework Block Details page
Allign copy button
…ansfers TX page: Token transfer and minting section improvements
…tokens-update Indexer performance update: Add skip_metadata flag for token if indexer failed to get any of [name, symbol, decimals, totalSupply]
Smart-Contract: remove comment with the submission date
rkachowski
approved these changes
Sep 3, 2021
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.
Looks fine, let's test the "skip token transfer if metadata is missing" against celo transfers on staging.
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.
W35 main features:
Block Details
andTransaction Details
pages