Skip to content

Webserver bug #416

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

Merged
merged 8 commits into from
Jul 13, 2020
Merged

Webserver bug #416

merged 8 commits into from
Jul 13, 2020

Conversation

archie94
Copy link
Collaborator

  • Plus a few improvements around dependency bump, lint warning fixes and code cleanups

Should fix #410

Please review and consider merging

Signed-off-by: Arka Prava Basu <[email protected]>
Signed-off-by: Arka Prava Basu <[email protected]>
- this flag prevented the dismissal of keyboard after an attempt
  to edit any EditTextPreference was made. We want the keyboard to
  dismiss after any edit for better UX

Signed-off-by: Arka Prava Basu <[email protected]>
- my work email got commited due to automatic signature from
  my work machine. Replace that with my personal email

Signed-off-by: Arka Prava Basu <[email protected]>
- less memory per instance + no implicit ref to parent class

Signed-off-by: Arka Prava Basu <[email protected]>
- helps in logging and better equality checks

Signed-off-by: Arka Prava Basu <[email protected]>
- add assertion to prevent Nullable warnings in lint

Signed-off-by: Arka Prava Basu <[email protected]>
- string representation of Date object is not allowed causing an
  IllegalStateException which prevented display of EventTriggers
- fixes issue guardianproject#410

Signed-off-by: Arka Prava Basu <[email protected]>
@n8fr8 n8fr8 merged commit e6f365b into guardianproject:master Jul 13, 2020
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.

Events empty on remote device
2 participants