Pause resume queue #586
Merged
Pause resume queue #586
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 8 commits
March 14, 2021 13:12
…ering, state of queue is behind
daltonfury42
requested changes
Mar 21, 2021
There was a problem hiding this comment.
Hey
Thanks a lot for the PR. I've left some initial comments
Let's get this much merged, we can handle the ribbon as a follow up PR. We could try see if this works well for us.
daltonfury42
requested changes
Mar 22, 2021
daltonfury42
requested changes
Mar 23, 2021
daltonfury42
approved these changes
Mar 23, 2021
There was a problem hiding this comment.
This PR was nicely done. @mmendiratta Thanks!
@maaverik LGTM
maaverik
approved these changes
Mar 23, 2021
|
@maaverik Let's merge this after the release so that we can iterate on it if required. |
|
Sounds good |
…to pause-resume-q
maaverik
added a commit
that referenced
this pull request
Apr 17, 2021
* Remove amplify backend (#524) * Async actions for Queue Status and My Queues API calls (#522) * Add gitter chat badge (#530) * Pick api base url from config. (#525) * Remove unused backend env amplify (#532) * Add LoadingStatus Component (#531) * Add LoadingStatus Component Upgrade storybook to 6.1.17 Fix LoadingIndicator stories * Pause resume queue (#586) * Merge fixes after pause queue (#598) * Add plus prefix to phone number, rename onClick to onSubmit (#600) * Fix Sonarqube code smells and lint errors (#599) * Fix Sonarqube code smells and lint errors * Address review comments - revert direct return and eslintrc * Some minor changes to add A2HS (#602) * Added size property to the icons & shortened name * file name changed * a square 144x144px icon of the logo * register the service worker to enable pwa * Modified phone icon on footer and ErrorHandler (#604) * Removed phone icon with outdated link * Added optional chaining in PageNotFound * Fix return home button when reaching from errorHandler * Intentional bug to check sentry logging * Customized sentry logs * Still experimenting with sentry * Trying another approach * Reverted intentional bugs and removed popup notifcation (#605) * Reverted intentional bugs and removed popup notifcation * Fix code style issues with ESLint * Make Contact Us on Navbar bit more bold to stand out (#609)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Just getting some feedback on this, currently has the ability to pause the queue from the admin side, and doesn't allow a user to join the queue until it gets resumed. Still needs the styling for the admin side.
Question: is there an SVG or some type of component created for the red
pausedbanner that goes over the share button. Shown in the design