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

Remove export archive from plugin header #2779

Merged
merged 2 commits into from
Apr 13, 2022

Conversation

mgdelacroix
Copy link
Contributor

@mgdelacroix mgdelacroix commented Apr 13, 2022

Summary

Follow up to #2776 to remove the export archive option from plugin mode. A redundant check on the API has been removed as well as cleanup.

Ticket Link

Related to #2672
Fixes #2780

@mgdelacroix mgdelacroix added the 2: Dev Review Requires review by a core committer label Apr 13, 2022
@mgdelacroix mgdelacroix requested review from sbishel and chenilim April 13, 2022 14:52
@mgdelacroix mgdelacroix requested a review from a team as a code owner April 13, 2022 14:52
Copy link
Collaborator

@sbishel sbishel left a comment

Choose a reason for hiding this comment

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

Just some tests that need updated, but the errors look expected

@mgdelacroix
Copy link
Contributor Author

Snapshot updated, thanks @sbishel!!

@mgdelacroix mgdelacroix merged commit 59b3870 into main Apr 13, 2022
@mgdelacroix mgdelacroix deleted the hide-export-archive-on-plugin branch April 13, 2022 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2: Dev Review Requires review by a core committer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

UX: Remove export all archives from plugin mode
2 participants