1221 Commits (d7f7ad64cc856bbc0ff9656c4a9109a7b4963ada)

Author	SHA1	Message	Date
Luke Imhoff	329aa3e627	Ensure order is preserved in Chain.hashes_to_addresses ... Fixes #843 Use order of hashes passed to `Chain.hashes_to_addresses` to order returned `Address`es.	6 years ago
Luke Imhoff	32380ba1e9	prometheus_ecto ... Ecto query duration metrics for Prometheus.	6 years ago
Felipe Renan	f9069b2871	Ignore the burn_address for ERC-721 in the query	6 years ago
Lokraan	10e17d34db	Run mix gettext.extract --merge ... Run mix format. Conform to circle ci tests. Format, tests, gettext. Removed unecessary tab in `address_validation_controller.ex` Fixed tests in `chain_text.exs` (address_to_validation_count/1 & get_blocks_validated_by_address/2). Ran `mix format` and `mix gettext.extract --merge Changed == to === in address.js to appease the all powerful eslint.	6 years ago
Lokraan	d4aedd08f4	Fix queries. ... Update chain queries and created tests for mentioned queries.	6 years ago
Lokraan	92f99cac22	Ran mix format on code. ... Confirm to CircleCi tests. Was failing due to not updating gettext, javascript lint standards, and an accidental edit in `block_controller.ex` where I added `:block => :required` to `necessity_by_association`. I also updated the query in `Explorer.Chain.address_to_validation_count/1` in order to make it more functional (use built in Ecto queries instead of hand written SQL query.) Conform to code review comments. Instances of "Blocks #{gettext("Validated")}" -> gettext("Blocks Validated") The class for the validation link in the mobile menus was "nav-link" instead of "dropdown-item" so I changed it. Ran `mix gettext.extract --merge` in order to update the gettext("Blocks Validated") Conform to code review. Added `active` classes to the corresponding links on mobile navs. Removed redundant if statements when in the page that required the content. Created a function in `address_view.ex` called `validator?` that accepts the number of validations and returns true if greater than 0. Replaced `if @validation_count > 0` with `if validator?(@validation_count`. "Please add the active class to the dropdown item." Added the `active` class where the page was active on for mobile and desktop navs. Also replaced `nav-item` with `dropdown-item` on the mobile navs where it was necessary.	6 years ago
Lokraan	cbb88f18fe	Validations are now viewable. ... Validation count, only shows validations if has them. Validated -> Blocks Validated, gettext(Validated)	6 years ago
Sebastian Abondano	07a0de6aed	account#tokenlist API endpoint ... Why: * For API users to be able to get a list of tokens owned by a given address. Example usage: ``` /api?module=account&action=tokenlist&address={addressHash} ``` * Issue link: https://github.com/poanetwork/blockscout/issues/814 This change addresses the need by: * Creating `Explorer.Etherscan.list_tokens/1` to fetch tokens owned by a given address. * Creating `API.RPC.AddressController.tokenlist/2` action to process requests to `account#tokenlist` API endpoint. * Editing `API.RPC.AddressView` to render token lists as required. * Adding documentation data for the new API endpoint mentioned above. Documentation data lives in `BlockScoutWeb.Etherscan`.	6 years ago
Amanda Sposito	92a241f4ac	Add 'Token Inventory' query	6 years ago
Luke Imhoff	4a1e75a87b	/blocks/:id works with number or hash now ... * Override param to be "hash_or_number" instead of "id" * If "hash_or_"number" starts with "0x", treat as hash; otherwise, treat as number like before.	6 years ago
Luke Imhoff	49f99b1400	Search for non-consensus blocks by hash	6 years ago
Luke Imhoff	12f789a96d	Use hash for Block conflict_target ... As the consensus block and zero or more non-consensus (uncle) blocks can have the same number now, number is no longer unique and not a good conflict_target, so switch to hash.	6 years ago
Luke Imhoff	07bd999e35	update_block_second_degree_relations ... After blocks are inserted, mark any rows in block_second_degree_relations as uncle_fetched_at where the uncle_hash matches the blocks hashes, so that stream_unfetched_uncle_hashes will no longer return those blocks.	6 years ago
Luke Imhoff	483ea2339f	Explorer.Chain.stream_unfetched_uncle_hashes ... Streams unique uncle hashes from block_second_degree_relations.	6 years ago
Luke Imhoff	818f84a424	Transaction.Fork ... A Transaction.Fork is a Transaction recorded in a non-consensus block, such an uncle.	6 years ago
Luke Imhoff	4b140c87df	Block in the consensus chain marked consensus true	6 years ago
Luke Imhoff	96d5aea311	Block.SecondDegreeRelation ... The database table block_second_degree_relations contains links between the `uncle_hash` of the uncle and the `nephew_hash` of the block that said `uncle_hash` was an uncle. It also has a `uncle_fetched_at` that is `null` until the `uncle_hash` is confirmed as fetched in the database. This does not contain such a fetcher.	6 years ago
Stamates	ff5d4d0163	Wireup live update of pending transactions	6 years ago
Lucas Narciso	428e674667	Handle more errors types at Smart Contract Reader ... Errors may pop up that do not comply with the current expect API: they may not include a :message key in its formation. When they did not, the exception handler was throwing an exception.	6 years ago
Felipe Renan	a8b185a865	Add Token Transfers's tab in the Address' page.	6 years ago
Felipe Renan	e4d4154fc7	Build query to fetch transactions with token transfers ... The query is scoped by the given address hash and the token hash. So, it will considers transactions that have token transfers so that the given address hash is the `to` or the `from` from a token transfer.	6 years ago
Gustavo Santos Ferreira	958a277bad	create Address.TokenCard to do the querying of tokens from an address	6 years ago
Gustavo Santos Ferreira	7e8974c4fb	alter query to bring balances and count transfers all at once	6 years ago
Gustavo Santos Ferreira	172a2cd6bc	make sure pagination disregards case	6 years ago
Sebastian Abondano	36d3d95e68	Improve transaction#getstatus API endpoint ... Why: * To increase the accuracy of `transaction#getstatus` API endpoint results. * Issue link: https://github.com/poanetwork/blockscout/issues/743 This change addresses the need by: * Editing `transaction#getstatus` API endpoint to use the recently added `Chain.transaction_to_status/1` function. This improves the accuracy of the errors by providing a reason of `awaiting internal transactions` for transactions with a failed status but for which the final "real" error message will only become available after processing the internal transactions. * Deleting `Explorer.Etherscan.get_transaction_error/1`. It's no longer needed.	6 years ago
Sebastian Abondano	6be5bc81b7	Derive status from internal transaction at index 0 ... Why: * Issue #757 was created after we noticed a few constraint violations for the `error` column in the `transactions` table. The constraint violations came about as we attempted to update transactions with a successful status but some value for the error field. The constraint is violated in this scenario because we expect successful transactions to not have an error. [EIP 658](https://github.com/ethereum/EIPs/pull/658), which apparently is also called [EIP 98](https://github.com/ethereum/EIPs/issues/98#issuecomment-318670322) , says that the status should be set to 1 (success) if the outermost code execution succeeded (internal transaction with index 0), or it should be set to 0 (failure) if the outermost code execution failed. We're currently deriving the status from the last internal transaction when in fact we should be deriving it from the first internal transaction. * Issue link: https://github.com/poanetwork/blockscout/issues/757 This change addresses the need by: * Editing `Explorer.Chain.Import.update_transactions/2` to set the error for a transaction equal to the error, if any, of the internal transaction with index 0. * Editing `Explorer.Chain.Import.update_transactions/2` to set a transaction's status as successful if it's internal transaction at index 0 doesn't have an error, and as failed if it does. This only applies to pre-Byzantium and Ethereum Classic, for which transaction status is not set from receipts.	6 years ago
Stamates	e000921433	Remove FakeAdapter for count estimation and just assert that an integer is returned from count estimation functions.	6 years ago
Stamates	cfac04336d	Remove pagination and return top 250 addresses by balance	6 years ago
Stamates	6ee50a6993	Estimated address count of addresses with balance > 0 using EXPLAIN rows estimate	6 years ago
Stamates	c060b86527	Wireup top_addresses page	6 years ago
Stamates	af6976317d	Use EXPLAIN in count query to speed up and get estimated row results	6 years ago
Stamates	4598eba91f	Remove transaction table joins from address_transaction_count query and count only distinct hashes	6 years ago
Tim Mecklem	d5d039a78d	Add address transaction count query back and include token transfers	6 years ago
Tim Mecklem	285fa9fa1e	Run transaction fetch queries separately per address_field ... This results in a much better query plan that takes advantage of some multi-column indexes coming soon to a GitHub near you.	6 years ago
Stamates	c515d765c1	Split address transactions query into separate query/fetches and combine the results due to postgres not selecting optimized query plan consistently	6 years ago
jimmay5469	a321b90878	Add full details to internal transaction tiles ... Co-authored-by: Stamates <stamates@hotmail.com>	6 years ago
Sebastian Abondano	8da1345d51	Add transaction#getstatus API endpoint ... Why: * For API users to be able to get the error for a given transaction hash. Example usage: ``` /api?module=transaction&action=getstatus&txhash={transactionHash} ``` * Issue link: https://github.com/poanetwork/blockscout/issues/138 This change addresses the need by: * Adding `Explorer.Etherscan.get_transaction_error/1` to get the error for a given transaction hash. * Adding `API.RPC.TransactionController.getstatus/2` action to process requests made to the `transaction#getstatus` API endpoint. * Editing `API.RPC.TransactionView` to render transaction errors ('status') as required. * Adding `transaction#getstatus` API endpoint to the API docs page. Docs data lives in `BlockScoutWeb.Etherscan`.	6 years ago
Luke Imhoff	b92c8511f5	Show transaction error in UI	6 years ago
Luke Imhoff	102f467948	Denormalize last internal transaction error to transaction ... Allow UI and APIs to get the overall transaction error easily when status is `:error` without have to preload all the internal transactions.	6 years ago
Luke Imhoff	b32f263eda	Allow status to be nil until internal transactions are indexed ... For pre-Byzantium or Ethereum Classic (which never adopted EIP-658), the status in receipts will be `nil`, but we still want to ensure the status is eventually set, so when setting internal transactions, in addition to setting `internal_transactions_indexed_at` and `created_contract_address_hash`, the `status` will be updated. * If the `status` is already set, leave it as is * If the `status` is not set * If the last internal transaction for a transaction has an error, `status` is `:error` * If the last internal transaction for a transaction has no error, `status` is `:ok`	6 years ago
Delweng Zheng	de1a101c8c	fix(apps): adjust gas_used to Decimal	6 years ago
Amanda Sposito	3244113be4	Increase timeout at the `Repo.transaction` level at the 'Indexer.TokenBalance.Fetcher'	6 years ago
Felipe Renan	53cb6de504	Handle multiple values when reading Smart Contract functions ... We received the following error when a function executed by the contract returned multiple values: ``` an exception was raised: ** (FunctionClauseError) no function clause matching in Explorer.SmartContract.Reader.new_value (explorer) lib/explorer/smart_contract/reader.ex:272: Explorer.SmartContract.Reader.new_value( %{"name" => "narcoName", "type" => "string"}, ["Justin Sun", 0, 0, [8, 6, 9, 2, 2, 37], [0, 0, 0, 0], "m0310100904381635021206170608010621080706", 6, 33, [1533368345, 1533331805, 1533339540, 1533333875, 0, 0], 277, [72, 29, 59, 0, 1, 0, 2, 73, 0] ] ) (explorer) lib/explorer/smart_contract/reader.ex:268: anonymous fn/3 in Explorer.SmartContract.Reader.link_outputs_and_values/3 ``` As the returned values match with the ABI outputs, we are relying on output's index to fetch the correct value for each output. This commit also fixes the template to deal with different results, such as: `[8, 6, 9, 2, 2, 37]`, that is an array of numbers but the Phoenix view considers it as a char-list.	6 years ago
Sebastian Abondano	ed66f9f87e	Orders account#getminedblocks by block number ... Why: * To match Etherscan's API we want to ensure blocks returned by the `account#getminedblocks` API endpoint are ordered by descending block number. We noticed this wasn't the case when looking into a couple of flaky tests on CI. This commit should also make sure the tests don't randomly fail any more. These are the two tests: - test/block_scout_web/controllers/api/rpc/address_controller_test.exs:1674 - test/explorer/etherscan_test.exs:1014 * Issue link: n/a This change addresses the need by: * Editing `Explorer.Etherscan.list_blocks/1` to order blocks by descending block number. * Related tests were edited according to the change above.	6 years ago
Alex Garibay	0a922bb1a3	Display primary address name when available (#610) ... * Display primary address name in overview tile * Display primary address name in links Co-authored-by: Luke Imhoff <Kronic.Deth@gmail.com>	6 years ago
Gustavo Santos Ferreira	605d28cb4d	preload token in queries that need it	6 years ago
Igor Florian	d460fb8d50	Deal with :bad_gateway errors while querying contracts functions ... We were counting on a specific error standard from ethereum_jsonrpc for formatting the errors from the SmartContract.Reader and we ended up getting an exception because we were getting a different error. There are at least two different error standards returned from ethereum_jsonrpc: 1. ```elixir { :ok, [ %{ id: id, jsonrpc: "2.0", error: %{ code: "12345", message: "<message>" } } ] } ``` 2. ```elixir {:error, {:bad_gateway, "<url>"}} ``` We're now expecting the :bad_gateway error so we can format it properly.	6 years ago
Felipe Renan	d9ff36c00d	Fix token balances' query ... This query was considering the next-to-last token balance when the last one had the value 0. In order to this work properly, I'm moving the query that gets token balances from the last block to a sub-query and moving the condition to consider just values greater than 0 to the main query. This way we are sure that we always are considering the last block and we are skipping the ones that have the value equal to 0.	6 years ago
Felipe Renan	d0766c2017	Rafactor to count the holders from token balances instead of token transfers	6 years ago
Felipe Renan	b856eed594	Build query to fetch the holders	6 years ago