-
Notifications
You must be signed in to change notification settings - Fork 33
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
feat: verify networkselector #550
Conversation
✔️ Deploy Preview for tradetrust-mainnet ready! 🔨 Explore the source changes: 25e4484 🔍 Inspect the deploy log: https://app.netlify.com/sites/tradetrust-mainnet/deploys/621c7bbfa8c0bb0007925a77 😎 Browse the preview: https://deploy-preview-550--tradetrust-mainnet.netlify.app/ |
✔️ Deploy Preview for tradetrust-mainnet ready! 🔨 Explore the source changes: 11bc0ee 🔍 Inspect the deploy log: https://app.netlify.com/sites/tradetrust-mainnet/deploys/622b025afd3a5600081afeba 😎 Browse the preview: https://deploy-preview-550--tradetrust-mainnet.netlify.app |
993e256
to
98011d7
Compare
98011d7
to
7e51b5a
Compare
Summary
Addition of Network Selector to Verify Page, and removing placement of network selector on top bar
Changes
Checks if uploaded file can be parsed by JSON Parser - outputs error if can't be parsed
Checking on OA Document, to see if NetworkSelected matches document's network field.
Tests for the above
Issues
Move Network Selector to Verify Page + UI Updates
https://www.pivotaltracker.com/story/show/181134373
Remove NetworkSelector from AppContainer
https://www.pivotaltracker.com/story/show/181134371