implements demo-ui #1410
Merged
implements demo-ui #1410
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
Westlad
approved these changes
May 4, 2023
Westlad
added
the
One more approval needed
IlyasRidhuan
approved these changes
May 24, 2023
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.
What does this implement/fix? Explain your changes.
This PR create a small react app called demo-ui, intension is demo-ui will be used to test contracts deployed in mumbai testnet.
for now: it has been test against local ganache instead of mumbai testnet deployments
fixes #1385
pending work:-
find-unused-exportsto make demo-uinpm run lintwork. Solution is to find better module than find-unused-exports to make it work for both node.js and react code.Does this close any currently open issues?
What commands can I run to test the change?
To run demo-ui:-
Any other comments?