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

Add related content information to the delete asset warning #1390

Closed
sivaschenko opened this issue May 26, 2020 · 2 comments · Fixed by #1401
Closed

Add related content information to the delete asset warning #1390

sivaschenko opened this issue May 26, 2020 · 2 comments · Fixed by #1401
Assignees
@ProkopovVitaliy
Labels
Frontend/UI Priority: P2 Should be fixed as early as possible Progress: good first issue Good for newcomers
Milestone
2.1.0

Comments

@sivaschenko
Copy link
Member

sivaschenko commented May 26, 2020 •
edited
Loading

Story

#1310

Preconditions

  1. Install Magento with Adobe Stock Integration
  2. Enable Enhanced Media Gallery

Steps to reproduce

  1. Open Media Gallery
  2. Upload an image
  3. Insert the image to Page/Block/Product/Category wysiwyg field
  4. Delete the uploaded image
  • Selecting the image and clicking Delete button
  • Opening view details and clicking Delete button
  • Opening context menu for the image (click on 3 dots) and selecting Delete option

Expected result

The warning message appears with the information that the image is used in the content

image
@sivaschenko sivaschenko added the Progress: good first issue Good for newcomers label May 26, 2020
@sivaschenko sivaschenko added this to the 2.1.0 - Part 1 milestone May 26, 2020
@m2-assistant
Copy link

m2-assistant bot commented May 26, 2020

Hi @sivaschenko. Thank you for your report.
To help us process this issue please make sure that you provided sufficient information.

Please, add a comment to assign the issue: @magento I am working on this

@ProkopovVitaliy
Copy link
Contributor

@magento I am working on this

@ProkopovVitaliy ProkopovVitaliy mentioned this issue May 30, 2020
Merged
@lenaorobei lenaorobei linked a pull request Jun 4, 2020 that will close this issue
#1390 Add related content information to the delete asset warning #1401
Merged
@sivaschenko sivaschenko added the Priority: P2 Should be fixed as early as possible label Jun 5, 2020
gabrieldagama pushed a commit that referenced this issue Jun 15, 2020
Merge pull request #1401 from ProkopovVitaliy/#1390
5b97a20 
#1390 Add related content information to the delete asset warning
@chalov-anton chalov-anton mentioned this issue Jun 22, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ProkopovVitaliy ProkopovVitaliy

Labels
Frontend/UI Priority: P2 Should be fixed as early as possible Progress: good first issue Good for newcomers
Projects
None yet
Milestone
2.1.0
Development

Successfully merging a pull request may close this issue.

#1390 Add related content information to the delete asset warning
2 participants
@sivaschenko @ProkopovVitaliy