Skip to content

Conversation

@quang-le
Copy link
Member

@quang-le quang-le commented Dec 30, 2025

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

Related JIRA tickets : POLIO-2061

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)

Doc

jsdoc comments in new hooks/components

Changes

Add functionality to existing components:

  • LinkTo: enable using underlying overrideIcon prop
  • CampaignAsyncSelect: enable more parametrization (campaign type, category, on hold etc)

Add/move:

  • useSkipEffectUntilValue: move to core iaso codebase and update imports

  • useAsyncInitialState: convenience hook for useState when initial value is async (fetched)

  • IntegratedCampaignsFields: CampaignAsyncSelect + link to campaign wrapped in a component to keep JSX readable

  • reorganize folder structure a bit

  • Add IntegratedCampaignsFields to BaseInfoForm

How to test

  • Create new non-polio campaign and add a campaign to integrate to
  • Go to "parent" campaign: the integrated campaign should be visible in the list/widget
  • repeat with editing a campaign

Print screen / video

Screen.Recording.2025-12-30.at.14.15.07.mov

Notes

merges BLSQ/bluesquare-components#196

- TODO: replace obr name with ID to simplify validation
- replace serializerFields
- update tests
- extract WidgetPaperRow for reuse
- outline add integrated component
- align delete icons
- use better translations
- update blsq-comp
@quang-le quang-le changed the title POLIO-2062: integrate non polio to polio POLIO-2061: integrate non polio to polio Dec 30, 2025
@quang-le quang-le added the release Should be released in production at next deploy label Dec 30, 2025
@beygorghor beygorghor self-requested a review January 5, 2026 10:04
Copy link
Collaborator

@beygorghor beygorghor left a comment

Choose a reason for hiding this comment

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

this is working but why don't add one more tab for integrated campaigns instead of having it on base infos ?

Base automatically changed from POLIO-2055_integrated_campaigns_UI to develop January 6, 2026 14:07
@quang-le quang-le merged commit ad70956 into develop Jan 6, 2026
7 checks passed
@quang-le quang-le deleted the POLIO-2062_integrate_non_polio_to_polio branch January 6, 2026 14:24
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