Skip to content

Feature/751 short form application word count stage 2 - #760

Merged
Plebbaroni merged 5 commits into
mainfrom
feature/751-short-form-application-word-count-stage-2
May 19, 2026
Merged

Feature/751 short form application word count stage 2#760
Plebbaroni merged 5 commits into
mainfrom
feature/751-short-form-application-word-count-stage-2

Conversation

@Siya-Yuan

@Siya-Yuan Siya-Yuan commented May 11, 2026

Copy link
Copy Markdown
Member

Note

  • Fix the classnames

Changes

  • The word count limit is live and displayed for the applicants on short answer questions (if admins specified a word limit). The behaviour is enforced so that when the word count limit is reached, the user cannot type another word.

@Siya-Yuan
Siya-Yuan requested a review from Plebbaroni May 11, 2026 10:59

@Plebbaroni Plebbaroni 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.

Great work on this PR, just some a minor issue relating to classnames. Ty!

@Siya-Yuan

Copy link
Copy Markdown
Member Author

I have now fixed the classnames. Thank you!

@Siya-Yuan
Siya-Yuan requested a review from Plebbaroni May 18, 2026 06:46
@Plebbaroni
Plebbaroni merged commit 1e17a3a into main May 19, 2026
2 checks passed
@Plebbaroni
Plebbaroni deleted the feature/751-short-form-application-word-count-stage-2 branch May 19, 2026 02:55
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