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

Devtool page in browser #486

Merged
merged 24 commits into from
Nov 17, 2023
Merged

Devtool page in browser #486

merged 24 commits into from
Nov 17, 2023

Conversation

PolpEdu
Copy link
Contributor

@PolpEdu PolpEdu commented Nov 7, 2023

This PR implements a Devtool page in browser containing all RPC requests made in the browser (see image below for an example)

image

Update

image

naps62 and others added 10 commits November 3, 2023 13:18
Apparently release builds were failing to run on some systems. For
example, when running AppImage on linux, the previously used call the
get resource paths would point to a temp dir in `/tmp`. This would lead
to errors attempting to create the DB file.

The new approach points to `~/.config/iron-wallet`.

Since this is a breaking change, I also took this opportunity to rename
the identifier from `com.naps62.iron` to `iron-wallet`, since that's
where the directory name will come from.

I'm also happy with the idea of not making `1.0` be a very big deal, and
instead follow semver religiously. So there we go, we reached 1.0!
Updating dev tool extension
@PolpEdu
Copy link
Contributor Author

PolpEdu commented Nov 8, 2023

Also added RPC responses to appear in the DevTool & improved Request organization (so that the responses are always attached to their request)
image

@naps62
Copy link
Member

naps62 commented Nov 15, 2023

Large refactor and re-design:
image

@naps62 naps62 added the feature label Nov 15, 2023
@naps62 naps62 merged commit ca7e4e4 into ethui:main Nov 17, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants