-
Notifications
You must be signed in to change notification settings - Fork 93
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
Scan from user registered block #549
Conversation
✅ Deploy Preview for jolly-shaw-20fe62 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
a3c0cca
to
3bb43ed
Compare
b6ea200
to
b58e13d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good so far @garyghayrat, this will be a big UX boost. Identified some edge cases in initial testing.
82d981c
to
0faa368
Compare
62605e0
to
1b96c9c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Working, for me
1b96c9c
to
5725cc9
Compare
- Fetch all announcements in advanced mode w private key
5725cc9
to
80c8bf1
Compare
Coverage after merging scan-from-user-registered-block into master will be
Coverage Report |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work on this @garyghayrat. Will be a big UX improvement for users.
One of the improvements listed in #266