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

Sandstorm: Countdown timer alert message missing #29

Open
sreffler opened this issue Feb 27, 2019 · 3 comments
Open

Sandstorm: Countdown timer alert message missing #29

sreffler opened this issue Feb 27, 2019 · 3 comments
Labels
bug (med priority) Something isn't working (somewhat important to fix) Sandstorm Sandstorm screen

Comments

@sreffler
Copy link

Expected result: On Sandstorm screen load, a countdown timer of 15 seconds is started. When the timer has reached 3 seconds, an alert message displays to signal to the user the end of the Sandstorm period. The remaining value of the timer is displayed in the alert message until the timer reaches 0. When the timer reaches 0, the user is advanced to the teleop screen and the alert message is cleared.

Actual result: No message displays. When timer expires, app crashes.

Note: Ashna believes we cannot use a toast notification for this, as it creates a non-clickable screen while the notification is visible. We agreed that it would be okay to do hide/show text that says "Auton starting soon," as long as all other functionality works as expected.

@sreffler sreffler added bug (med priority) Something isn't working (somewhat important to fix) Sandstorm Sandstorm screen labels Feb 27, 2019
@aschimmichanga
Copy link
Contributor

works as expected (yellow warning displayed)

@sreffler
Copy link
Author

sreffler commented Mar 9, 2019

"Starting soon" is displayed at all times during Sandstorm, instead of appearing 3 seconds before the end of the countdown timer.

Best solution for Mt. Olive is to hide this UI element altogether.

@sreffler sreffler reopened this Mar 9, 2019
@sreffler
Copy link
Author

Was removed for Mt. Olive, still needs to be addressed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug (med priority) Something isn't working (somewhat important to fix) Sandstorm Sandstorm screen
Projects
None yet
Development

No branches or pull requests

2 participants