Skip to content

Conversation

@Crebert08
Copy link
Contributor

What problem is this PR solving? Explain here in one sentence.

Related JIRA tickets : https://bluesquare.atlassian.net/browse/IA-2240

Self proofreading checklist

  • Did I use eslint and ruff formatters?
  • Is my code clear enough and well documented?
  • Are my typescript files well typed?
  • New translations have been added or updated if new strings have been introduced in the frontend
  • My migrations file are included
  • Are there enough tests?
  • Documentation has been included (for new feature)

Changes

Added the useDeleteTableRow hook into the useDelete hooks of these following components:
-entitytypes
-forms
-Budget
-Grouped Campaigns
-Teams
-planning
-OrgUniTypes Groups
-OrgUniTypes

I didn't make any changes in these following components yet:
-useDeleteBeneficiary
-useDeleteAssignements
-useDeleteAction
-useDeleteInstance
-useDeleteWorkflowChange
-useDeleteWorkflowFollowup
-useDeleteWorkflowVersion

fix: empty instance pop up

Refs: IA-2240

Note that the Jira reference is preceded by a line break.

Both the line break and the Jira reference are entered in the Add an optional extended description… field.

@Crebert08 Crebert08 requested a review from quang-le December 10, 2025 12:47
@quang-le quang-le added the postrelease Should be merged just after the release label Dec 12, 2025
Copy link
Member

@quang-le quang-le left a comment

Choose a reason for hiding this comment

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

See comments. Not all might require a change, but please answer:

  • TableWithDeepLink ifstatement
  • Inconsistent queryKey invalidation
  • missing queryKeys in invalidation

@Crebert08 Crebert08 requested a review from quang-le December 18, 2025 14:40
@quang-le quang-le added release Should be released in production at next deploy and removed postrelease Should be merged just after the release labels Dec 24, 2025
@Crebert08 Crebert08 merged commit 96f9a9e into develop Jan 6, 2026
7 checks passed
@Crebert08 Crebert08 deleted the IA-2240-refactor-all-useDelete-hooks branch January 6, 2026 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release Should be released in production at next deploy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants