Skip to content
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

Implemented Transaction Details Page #109

Merged
merged 2 commits into from Oct 21, 2019

Conversation

@frol
Copy link
Member

frol commented Oct 15, 2019

Resolves #91

Notes:

  1. Transaction fees are not currently exposed via RPC, so we cannot show them yet. nearprotocol/nearcore#1299
  2. Action result and logs are not implemented yet, this is the only item left to be implemented.
  3. Mobile is ok, but we should definitely get back to it one day to review mobile readiness (including performance and network traffic size optimization) #111
  4. Tests are coming
  5. Links to the transaction details page from the dashboard, list of transactions, block details, account details pages work fine (in fact, those pages reuse the same component, so it was just a change in a single place)
  6. Render.com deployment won't work for this PR as it introduces a change on the proxy-backend.

Here is an example of a direct link to a transaction: https://near-explorer-frontend-pr-109.onrender.com/transactions/3dYfigPo3FGTb9rN2KhyXTXWrPsYaRDK69FTHW9So9Ff

image

@render

This comment has been minimized.

Copy link

render bot commented Oct 15, 2019

@frol frol force-pushed the feature/transaction-details-page branch from 9f10144 to b1cb153 Oct 18, 2019
@frol frol changed the title WIP: Implemented Transaction Details Page Implemented Transaction Details Page Oct 18, 2019
@frol frol requested review from deacix and kcole16 Oct 18, 2019
@frol frol merged commit 36a6ba6 into master Oct 21, 2019
2 of 3 checks passed
2 of 3 checks passed
Frontend unit testing (ubuntu-18.04, 10)
Details
Frontend e2e testing (ubuntu-18.04, 10)
Details
Backend testing (ubuntu-18.04, 10) Backend testing (ubuntu-18.04, 10)
Details
@frol frol deleted the feature/transaction-details-page branch Oct 27, 2019
deacix added a commit that referenced this pull request Oct 31, 2019
…search-box-99

* commit 'f6e22c0b41ec1852b04b5e8165d501a2873049f2':
  Implemented multinet support (resolves #110)
  Use a non-released version of nearlib with chunks API support (resolves #115)
  Resolved the compatibility issues with nearcore 0.4 (resolves #115)
  Fixed the double-reversed ordering issue on the Transactions list page
  Implemented Transaction Details Page (#109)
  Fixed actions ordering
  Feature/e2e (resolves #89)

# Conflicts:
#	frontend/package.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.