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

TX details: support switching between hex and base64 data display #904

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

csillag
Copy link
Contributor

@csillag csillag commented Sep 28, 2023

.. as requested by @matevz

More design feedback is needed by @donouwens .

@github-actions
Copy link

github-actions bot commented Sep 28, 2023

Deployed to Cloudflare Pages

Latest commit: 0720c0bb97bfd0994ddaceba139bd6c685daa101
Status:✅ Deploy successful!
Preview URL: https://3affa180.oasis-explorer.pages.dev

@csillag csillag force-pushed the csillag/data-format-switch branch from 61ba33b to 2f11454 Compare September 28, 2023 12:18
@matevz
Copy link
Member

matevz commented Sep 28, 2023

I propose we use the existing Oasis<->ETH toggle for switching between Oasis and ETH format. The intuition is the following:

  • Ethereum tooling use hex, so you won't come far with Base64 without a 3rd party decoder
  • Oasis tooling (e.g. Oasis CLI, oasis-node logs) use Base64 format everywhere

@csillag csillag force-pushed the csillag/data-format-switch branch from 2f11454 to 0720c0b Compare January 27, 2024 12:44
@csillag
Copy link
Contributor Author

csillag commented Jan 27, 2024

A friendly reminder: this is still waiting for feedback from @donouwens

@donouwens
Copy link
Collaborator

@csillag this makes sense to me so let's make it happen, please. Do you need any visuals from me on this?

@csillag
Copy link
Contributor Author

csillag commented Jan 29, 2024

Could you please check the current implementation?

@csillag
Copy link
Contributor Author

csillag commented Jan 30, 2024

@donouwens I think we should discuss this a bit more, since after I implemented it, @matevz has explained that he has actually meant something different...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants