Skip to content

Validate user input - #937

Open
ignatiusm wants to merge 27 commits into
devfrom
891-validate-input
Open

Validate user input#937
ignatiusm wants to merge 27 commits into
devfrom
891-validate-input

Conversation

@ignatiusm

@ignatiusm ignatiusm commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Resolves #891

  • Refactors application/common/types to make type constraints more DRY
  • Adds common functions for managing user input fields in both Admin and Participant portals
  • Adds tests to verify that invalid input is not accepted or saved to DB

ignatiusm added 27 commits July 29, 2026 15:46
@ignatiusm
WIP start dry type constraint refactor
7f6c7bd
@ignatiusm
WIP start dry type constraint refactor
93944be
@ignatiusm
Merge branch '891-validate-input' of github.com:Garvan-Data-Science-P…
5ff96f6 
…latform/ctrl into 891-validate-input
@ignatiusm
Refactor user types
5be124d
@ignatiusm
Make survey and study types more dry
2ef5a8d
@ignatiusm
Make settings types more dry
9090112
@ignatiusm
Make participant types more dry
904b747
@ignatiusm
Make organisation types more dry
d67cd8e
@ignatiusm
Make mailer types more dry
342f0eb
@ignatiusm
Make integration types more dry
ba0c0db
@ignatiusm
Make auth types more dry
95bab83
@ignatiusm
Make survey types more dry
31110f5
@ignatiusm
Tidy up regex
73b18c2
@ignatiusm
Remove illegal character from backend integration/Studies test
f34eafc
@ignatiusm
Remove illegal character from backend integration/Dependents test
7e47293
@ignatiusm
Get integration tests passing
8844012
@ignatiusm
Get auth and profiles tests passing
1d6b640
@ignatiusm
Move REGEX to commonTypes so all in one place
459e7aa
@ignatiusm
Use original mobile regex pattern (created issue for country code)
bb436f4
@ignatiusm
Get user-client tests passing before adding new tests
0e085e5
@ignatiusm
Get admin-client tests passing before adding new tests
60bb6f5
@ignatiusm
Fix email invite ExternalID minLength bug
061076f
@ignatiusm
Fix failing Auth tests with country code
2344f28
@ignatiusm
Add backend tests for invalid participant and admin details
ac12443
@ignatiusm
Add user-client cypress test for xss in registration
f3c77ad
@ignatiusm
WIP Adds xss tests to user-client profileEdit pages
0bfd65c
@ignatiusm
Adds xss tests to user-client profileEdit pages
e7a6721
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ignatiusm