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

[NV-3024] [NV-2803] 📡 Add warning modal on deactivate primary integration #4507

Open
davidsoderberg opened this issue Oct 12, 2023 · 4 comments
Assignees

Comments

@davidsoderberg
Copy link
Contributor

davidsoderberg commented Oct 12, 2023

What?

When the user deactivates the integration that is primary, we should warn him about the consequences (the primary flag will be deleted). And if he still decides about the action we will show the "Select Primary Provider modal" as it is today.

Why? (Context)

The primary flag on the integration is super important. During the sending phase, it decides whether the integration will be used to send the message.

Definition of Done

  • Display hint on hover over the disable/active toggle
  • Display a warning modal when the user deactivates the primary integration
  • If user decides to proceed, show the "Select Primary Provider modal" like it is today

Designs:

Created provider instance - Integration store (Figma)

Primary provider change caused by applying condition - Integration store (Figma)

Create instance / Step 8 - Integration store (Figma)

From SyncLinear.com | NV-2803

NV-3024

@davidsoderberg davidsoderberg changed the title [NV-2803] 📡 Add warning modal on deactivate primary integration [NV-3024] [NV-2803] 📡 Add warning modal on deactivate primary integration Oct 12, 2023
@michaldziuba03
Copy link
Contributor

@LetItRock @davidsoderberg you can assign me

@LetItRock
Copy link
Contributor

@LetItRock @davidsoderberg you can assign me

@michaldziuba03 sure, thanks! 🙌

@daemonX10
Copy link

assign me

@deep-yad
Copy link

please assign it to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants