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

make get actions trigger just if modal is shown #1788

Merged
merged 1 commit into from
Jan 30, 2023

Conversation

trilopin
Copy link
Contributor

Summary

Extracted from #1748. The rest of the original PR is being discussed more in-depth.

/plugins/playbooks/api/v0/actions/channels/

Make channel actions modal fetch action ONLY when is shown, not when it's rendered but hidden. Before this change, channel actions were fetched on every channel switch.

Ticket Link

Fixes (partially) #1731

Checklist

  • Telemetry updated
  • Gated by experimental feature flag
  • Unit tests updated

@trilopin
Copy link
Contributor Author

@lieut-data @crspeller I assigned again to you both because you already approved #1748.

@trilopin trilopin added the 2: Dev Review Requires review by a core committer label Jan 30, 2023
@lieut-data lieut-data added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core committer labels Jan 30, 2023
@lieut-data lieut-data merged commit 7e4aefd into master Jan 30, 2023
@lieut-data lieut-data deleted the fix-overfech-actions-modal branch January 30, 2023 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants