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

colw/Windows Nano Warning #2912

Merged
merged 8 commits into from
Aug 27, 2019
Merged

colw/Windows Nano Warning #2912

merged 8 commits into from
Aug 27, 2019

Conversation

colw
Copy link
Contributor

@colw colw commented Aug 27, 2019

Closes #2672

Description:

First pass at windows detection and warning.

Sign in:
Screenshot 2019-08-27 at 11 40 12

ActionModal *visible on each screen
Screenshot 2019-08-27 at 11 40 52

Thank you! 🚀


For contributor:

  • Added changes entries. Run yarn changelog for a guided process.
  • Reviewed Files changed in the github PR explorer
  • Attach screenshots of the UI components on the PR description (if applicable)
  • Scope of work approved for big PRs

For reviewer:

  • Manually tested the changes on the UI

@colw colw changed the title First pass at windows detection and warning colw/Windows Nano Warning Aug 27, 2019
@colw colw marked this pull request as ready for review August 27, 2019 11:18
src/vuex/modules/session.js Outdated Show resolved Hide resolved
Co-Authored-By: Jordan Bibla <jbibla@gmail.com>
@jbibla
Copy link
Collaborator

jbibla commented Aug 27, 2019

amazing. thank you colin!

@faboweb faboweb merged commit 04f36cf into develop Aug 27, 2019
@faboweb faboweb deleted the colw/windows-nano-warning branch August 27, 2019 13:57
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.

Unable to claim rewards if amount of validators is more than 5
3 participants