Skip to content

Resolve "Bug - Cancel/Add addon modal is glitchy with many plans (more than 10)"#548

Merged
mnida merged 5 commits intomainfrom
LOT-566-cancel-subscription-modal-vug
Feb 13, 2023
Merged

Resolve "Bug - Cancel/Add addon modal is glitchy with many plans (more than 10)"#548
mnida merged 5 commits intomainfrom
LOT-566-cancel-subscription-modal-vug

Conversation

@glamboyosa
Copy link
Copy Markdown
Contributor

Closes

Closes LOT-566.

Description

This PR fixes the bug in LOT-566 where if you had >=10 plans in the subscription view of a plan, triggering the modal would cause it to flicker.

Testing

  • Log onto the app.
  • Go to customers. select one.
  • Attach plans till the total number is 10 or more.
  • Triggering the modal should not cause it to flicker.

@linear
Copy link
Copy Markdown

linear Bot commented Feb 13, 2023

LOT-566 Bug - Cancel/Add addon modal is glitchy with many plans (more than 10)

### Summary

Check slack I think we summarized it a bit in there.

I looked into it and I think I made some fixes but i think because we close and open the modal a lot sometimes it flickers and gets overloaded.

This is not great detail but more should be provided in the Slack bug report diego made.

@glamboyosa glamboyosa temporarily deployed to Development February 13, 2023 13:24 — with GitHub Actions Inactive
@glamboyosa glamboyosa temporarily deployed to Development February 13, 2023 13:24 — with GitHub Actions Inactive
@glamboyosa glamboyosa temporarily deployed to Development February 13, 2023 16:37 — with GitHub Actions Inactive
@glamboyosa glamboyosa temporarily deployed to Development February 13, 2023 16:37 — with GitHub Actions Inactive
@mnida mnida added this pull request to the merge queue Feb 13, 2023
Merged via the queue into main with commit 1342307 Feb 13, 2023
@diego-escobedo diego-escobedo deleted the LOT-566-cancel-subscription-modal-vug branch February 14, 2023 05:46
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.

2 participants