Skip to content

Conversation

@Suhas2109
Copy link
Contributor

This PR enhances the Release Edit Page by introducing the missing LinkedPackagesTab and enabling proper linkage and deletion of packages for a release.

  • Linked packages for releases can now be viewed, linked, and deleted directly from the edit page.
  • Shared modal (LinkPackagesModal.tsx) works dynamically for both projects and releases.
  • Deleting linked packages updates UI and payload correctly.

@Suhas2109 Suhas2109 force-pushed the fix/linkedPackagesTabAndDeleteAction branch from 759ff25 to 55da5c5 Compare October 15, 2025 15:06
@afsahsyeda afsahsyeda added needs code review needs codes review needs general test needs general test Do not merge don't merge labels Oct 16, 2025
@Suhas2109 Suhas2109 force-pushed the fix/linkedPackagesTabAndDeleteAction branch from 55da5c5 to a972260 Compare October 16, 2025 09:20
@akshitjoshii
Copy link
Contributor

Delete action button not working. Pls integrate the functionality. On clicking the delete button the pkg disappears from the table, it should ideally give a warning and then if the pkg is not linked to any project proceed with deletion.
image

@Suhas2109 Suhas2109 force-pushed the fix/linkedPackagesTabAndDeleteAction branch from 0c7c65f to 73fb38c Compare October 29, 2025 07:00
@Suhas2109
Copy link
Contributor Author

@akshitjoshii
The delete action functionality has been implemented.
Can you Please perform general testing now.

@akshitjoshii
Copy link
Contributor

Pls check the build is failing.

@akshitjoshii
Copy link
Contributor

Rebase this branch with the latest main.

@Suhas2109 Suhas2109 force-pushed the fix/linkedPackagesTabAndDeleteAction branch 2 times, most recently from a20e280 to 7545185 Compare November 6, 2025 15:20
@akshitjoshii
Copy link
Contributor

@Suhas2109
pls improve the error msg like it is implemented in Liferay.

It should be clear to the user like this.

image

@Suhas2109 Suhas2109 force-pushed the fix/linkedPackagesTabAndDeleteAction branch 4 times, most recently from 77f3a68 to cdcf4a4 Compare November 17, 2025 04:28
@Suhas2109
Copy link
Contributor Author

image

Hi @akshitjoshii
Updated the error message to match the Liferay.

@akshitjoshii
Copy link
Contributor

@Suhas2109 , Pls take care of delete action in the Edit package page as well. Also change the delete action warning text.

@Suhas2109 Suhas2109 force-pushed the fix/linkedPackagesTabAndDeleteAction branch 2 times, most recently from 79638ea to 0f5d2cb Compare November 18, 2025 05:28
@Suhas2109 Suhas2109 force-pushed the fix/linkedPackagesTabAndDeleteAction branch from 0f5d2cb to d379226 Compare November 18, 2025 10:34
@Suhas2109 Suhas2109 closed this Nov 20, 2025
@Suhas2109 Suhas2109 deleted the fix/linkedPackagesTabAndDeleteAction branch November 20, 2025 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Do not merge don't merge needs code review needs codes review needs general test needs general test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants