Skip to content

fix: enforcing char limit on survey name.#2202

Open
ranisa-gupta16 wants to merge 5 commits into
mainfrom
feat/enforce-survey-name-limit
Open

fix: enforcing char limit on survey name.#2202
ranisa-gupta16 wants to merge 5 commits into
mainfrom
feat/enforce-survey-name-limit

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-1382)

Description

Creating a survey enforces a name length limit (5–100 characters), but editing
the name afterwards (Actions → Edit name and description) did not enforce it.

This issue is fixed as part of this.
Doesn't allow more than 100 chars and less than 5.
Test cases updated and all passed.

You were able to do less than 5 char too before and save it , its fixed.

image Screenshot 2026-07-21 164410 image

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.

Signed-off-by: Ranisa Gupta <ranisa.tech@gmail.com>
@ranisa-gupta16 ranisa-gupta16 changed the title fix: enforcing char limit on survey tempalte names. fix: enforcing char limit on survey name. Jul 22, 2026
@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown

E2E ❌ failed

Result ❌ Suite failed
Artifacts Download e2e-artifacts-30231059343.zip
Run View workflow run

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

Updated for commit 690e87c

@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 👍

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