Skip to content

feat: designer - add warning in designer if field ID is updated and show records count - #2203

Open
ranisa-gupta16 wants to merge 10 commits into
mainfrom
feat-designer/add-warning-if-fieldID-changes-in-survey
Open

feat: designer - add warning in designer if field ID is updated and show records count#2203
ranisa-gupta16 wants to merge 10 commits into
mainfrom
feat-designer/add-warning-if-fieldID-changes-in-survey

Conversation

@ranisa-gupta16

@ranisa-gupta16 ranisa-gupta16 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Ticket

[<JIRA_TICKET>
](https://jira.csiro.au/browse/BSS-1361)

Description

Changing a field's ID on a survey that already holds records can orphan the data collected against the old ID. There was no warning at all — the rename just happened.

Now, if the survey has records, changing a Field ID (either by typing a new one or clicking the ⟳ sync button) opens a confirmation dialog first. It names the old and new ID, says how many records are affected, and offers Sync or Cancel. It warns but doesn't block — you can still go ahead.

Screenshots below of updated flow:
Screenshot 2026-07-22 164721
Screenshot 2026-07-22 164744
Screenshot 2026-07-22 164748

Checklist

  • I have confirmed all commits have been signed.
  • I have added JSDoc style comments to any new functions or classes.
  • I have added tests for new code if appropriate
  • Relevant documentation such as READMEs, guides, and class comments are updated.

…g it when necessary and let user action the next..

Signed-off-by: Ranisa Gupta <ranisa.tech@gmail.com>
Signed-off-by: Ranisa Gupta <ranisa.tech@gmail.com>
@ranisa-gupta16 ranisa-gupta16 changed the title Feat designer/add warning if field id changes in survey feat: designer - add warning in designer if field ID is updated and show records count Jul 22, 2026
Signed-off-by: Ranisa Gupta <ranisa.tech@gmail.com>
@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown

E2E ✅ passed

Result ✅ Suite passed
Artifacts Download e2e-artifacts-30514987720.zip
Run View workflow run

After downloading and extracting the .zip file, open artifacts/index.html (failures listed first).

Updated for commit 231e7bd

@PeterBaker0 PeterBaker0 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good overall. I think the missing piece is tracking which fields were originally in the design so that we don't show alerts and warnings for re-syncing of field IDs for fields which are new in that session.

Comment thread web/src/designer/components/Fields/BaseFieldEditor.tsx
Comment thread web/src/designer/components/Fields/BaseFieldEditor.tsx
…ning-if-fieldID-changes-in-survey

Signed-off-by: Ranisa Gupta <ranisa.tech@gmail.com>
Signed-off-by: Ranisa Gupta <ranisa.tech@gmail.com>
…ndetifiers

Signed-off-by: Ranisa Gupta <ranisa.tech@gmail.com>
Signed-off-by: Ranisa Gupta <ranisa.tech@gmail.com>
Signed-off-by: Ranisa Gupta <ranisa.tech@gmail.com>
Signed-off-by: Ranisa Gupta <ranisa.tech@gmail.com>
Signed-off-by: Ranisa Gupta <ranisa.tech@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants