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.

Sign up for GitHub

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

Jump to bottom

Feature/116 implement delete from server #224

Merged
merged 2 commits into from
Nov 1, 2024
Merged

Feature/116 implement delete from server #224

merged 2 commits into from
Nov 1, 2024

Conversation

anasilva105
Copy link
Collaborator

PR details

What changes did you make? (Give an overview)

  • Implemented mcdev delete command into the extension. This command will only be executed for files or folders inside the asset-subtype.
  • Confirmation of deletion message is shown before running the command.
  • Code checks if delete action is valid for a certain metadata type. If not it shows an error message in the Output channel.

image
image

@anasilva105 anasilva105 added this to the 2.2.1 milestone Nov 1, 2024
@anasilva105 anasilva105 self-assigned this Nov 1, 2024
@github-actions github-actions bot added the enhancement New feature or request label Nov 1, 2024
@anasilva105 anasilva105 linked an issue Nov 1, 2024 that may be closed by this pull request
[FEATURE] Implement Delete From Server #116
Open
@anasilva105 anasilva105 merged commit e1e5074 into develop Nov 1, 2024
3 checks passed
@anasilva105 anasilva105 deleted the feature/116-implement-delete-from-server branch November 1, 2024 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JoernBerkefeld JoernBerkefeld JoernBerkefeld approved these changes

Assignees

@anasilva105 anasilva105

Labels
enhancement New feature or request feature
Projects
None yet
Milestone
2.2.1
Development

Successfully merging this pull request may close these issues.

[FEATURE] Implement Delete From Server
2 participants
@anasilva105 @JoernBerkefeld