Skip to content

Bugfix: remove and ignore assets#94

Merged
estebanmino merged 2 commits intomasterfrom
bugfix/remove-and-ignore-assets
Apr 10, 2019
Merged

Bugfix: remove and ignore assets#94
estebanmino merged 2 commits intomasterfrom
bugfix/remove-and-ignore-assets

Conversation

@estebanmino
Copy link
Copy Markdown
Contributor

This PR maintain changes made in #86 adding specific methods to remove and ignore assets, and maintaining remove assets methods before that PR.
The motivation of this is to maintain independent methods in AssetsController. This change produced that all removed assets would be immediately ignored without having an option to remove without ignore.

Copy link
Copy Markdown
Contributor

@brunobar79 brunobar79 left a comment

Choose a reason for hiding this comment

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

💯

@estebanmino estebanmino merged commit ea58b65 into master Apr 10, 2019
@estebanmino estebanmino deleted the bugfix/remove-and-ignore-assets branch April 10, 2019 18:27
mcmire pushed a commit to mcmire/core that referenced this pull request Jul 17, 2023
Keep original type when using hasProperty if defined
MajorLift pushed a commit that referenced this pull request Oct 11, 2023
* add remove and ignore assets methods

* tests
MajorLift pushed a commit that referenced this pull request Oct 11, 2023
* add remove and ignore assets methods

* tests
Mrtenz pushed a commit that referenced this pull request Oct 16, 2025
Bumps [normalize-url](https://github.com/sindresorhus/normalize-url) from 4.3.0 to 4.5.1.
- [Release notes](https://github.com/sindresorhus/normalize-url/releases)
- [Commits](https://github.com/sindresorhus/normalize-url/commits)

---
updated-dependencies:
- dependency-name: normalize-url
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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