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

feat(FileAction): Allow to set destructive flag #1076

Merged
merged 2 commits into from
Sep 11, 2024

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Sep 11, 2024

Makes the action be shown more prominently marked as destructive. Required for nextcloud/server#46732

Makes the action be shown more prominently marked as destructive.
Required for nextcloud/server#46732

Signed-off-by: Ferdinand Thiessen <[email protected]>
@susnux susnux added enhancement New feature or request 3. to review labels Sep 11, 2024
Copy link

codecov bot commented Sep 11, 2024

Bundle Report

Changes will increase total bundle size by 2.55kB (1.08%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
@nextcloud/files-esm 118.64kB 1.28kB ⬆️
@nextcloud/files-esm-cjs 120.02kB 1.28kB ⬆️

Copy link
Contributor

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

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

Good idea!

@skjnldsv skjnldsv merged commit 50465ed into main Sep 11, 2024
15 checks passed
@skjnldsv skjnldsv deleted the feat/allow-destructive-actions branch September 11, 2024 18:03
@@ -12,7 +12,7 @@ export enum DefaultType {
HIDDEN = 'hidden',
}

interface FileActionData {
export interface FileActionData {
Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, I missed this.
wasn't it a leftover from your other proposal @susnux ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants