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

IBX-8109: [ALW] folders management #1254

Merged
merged 7 commits into from
Jun 26, 2024
Merged

IBX-8109: [ALW] folders management #1254

merged 7 commits into from
Jun 26, 2024

Conversation

lucasOsti
Copy link
Contributor

@lucasOsti lucasOsti commented Apr 24, 2024 •
edited
Loading

🎫 Issue IBX-8109

Related PRs:

Description:

Added the ability to manage folders (Create, Delete, Rename, Move) from the tree level for Image Picker and Assets Library Widget.

For QA:

  1. Requires DXP tests (Image Picker) and a standalone app
  2. When confirming deletion, we use the standard confirmation modal, so currently there is a difference in design and implementation. The unification of this modal has a separate notification.

Documentation:

@lucasOsti lucasOsti mentioned this pull request Apr 24, 2024
Closed
@lucasOsti lucasOsti force-pushed the IBX-8109-ALW-folders-management branch 3 times, most recently from a9ab8dd to 60f0e7a Compare April 26, 2024 09:50
@lucasOsti lucasOsti changed the base branch from main to 4.6 April 26, 2024 09:52
@lucasOsti lucasOsti changed the title IBX-8109: alw folders management IBX-8109: [ALW] folders management Apr 26, 2024
@lucasOsti lucasOsti force-pushed the IBX-8109-ALW-folders-management branch from 60f0e7a to 2ffdc1f Compare May 7, 2024 08:42
@lucasOsti lucasOsti added Feature New feature request Ready for review labels May 7, 2024
@lucasOsti lucasOsti force-pushed the IBX-8109-ALW-folders-management branch from 50ca93a to bcb7ed2 Compare May 15, 2024 11:11
@lucasOsti lucasOsti requested review from dew326 and GrabowskiM May 15, 2024 14:05
@lucasOsti lucasOsti force-pushed the IBX-8109-ALW-folders-management branch 2 times, most recently from aca3172 to 3908a48 Compare May 28, 2024 09:28
@lucasOsti lucasOsti force-pushed the IBX-8109-ALW-folders-management branch from 3908a48 to c3216a0 Compare June 6, 2024 07:15
@bogusez bogusez self-assigned this Jun 6, 2024
@lucasOsti lucasOsti force-pushed the IBX-8109-ALW-folders-management branch from 5e90a3d to 863ce1a Compare June 20, 2024 08:19
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

This was referenced Jun 26, 2024
Closed
Closed
@webhdx webhdx merged commit a1447a7 into 4.6 Jun 26, 2024
22 checks passed
@webhdx webhdx deleted the IBX-8109-ALW-folders-management branch June 26, 2024 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GrabowskiM GrabowskiM GrabowskiM approved these changes

@tischsoic tischsoic tischsoic approved these changes

@dew326 dew326 dew326 approved these changes

@bogusez bogusez bogusez approved these changes

Assignees

@mikadamczyk mikadamczyk

@Nattfarinn Nattfarinn

@ciastektk ciastektk

@dew326 dew326

@ViniTou ViniTou

@bogusez bogusez

Labels
Feature New feature request QA approved
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@lucasOsti @GrabowskiM @tischsoic @dew326 @bogusez @mikadamczyk @Nattfarinn @ciastektk @webhdx @ViniTou