Skip to content

Conversation

@prateekshadas
Copy link

@prateekshadas prateekshadas commented Jul 29, 2020

  • Mac CI builds were taking a long time so fixed that
    To verify if this fixes the problem :
    There are multiple UI States to save the changes in Settings (when connecting/disconnecting Caffeine account ) and should follow following which changing Settings:

    1. Ok button - Should work same as Apply and should save the changes
    2. Cancel Button- Discard the changes
    3. Apply - Apply the changes
    4. Close the Settings Dialog Window with ❌ - Should open up the Message Box Dialog which prompts the user to save the unsaved changes
      • Yes -> save the changes ,
      • No - Discard changes and
      • Cancel - closes the Messages Box dialog.

This change is Reviewable

Copy link

@turbodavidson turbodavidson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link

@Xaymar Xaymar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

4 participants