Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

🙅 [PROBLEM]: Site ID must start with a letter and contain only letters, numbers and underscores #76

Closed
paddyroddy opened this issue Feb 21, 2024 · 0 comments · Fixed by #84
Closed

🙅 [PROBLEM]: Site ID must start with a letter and contain only letters, numbers and underscores #76

paddyroddy opened this issue Feb 21, 2024 · 0 comments · Fixed by #84

Comments

@paddyroddy
Copy link
Member

Problem Description

There is no validation of Site ID, so it complains if you don't follow the specification.

The id used to refer to this site (also used to generate database ids). The Site ID must start with a letter and contain only letters, numbers and underscores. No spaces or non-alphanumeric characters.

Acceptance Criteria

Related Work Items
Beta Give feedback
No tasks being tracked yet.
@paddyroddy paddyroddy mentioned this issue Feb 21, 2024
Merged
paddyroddy added a commit that referenced this issue Feb 21, 2024
@paddyroddy
Make site_name valid (#77)
163a8dd 
> The id used to refer to this site (also used to generate database
ids). The Site ID must start with a letter and contain only letters,
numbers and underscores. No spaces or non-alphanumeric characters.

Note, this does not address #76, but at least makes the `site_name` here
valid according to XNAT.
@p-j-smith p-j-smith mentioned this issue Mar 24, 2024
Merged
p-j-smith added a commit that referenced this issue Mar 25, 2024
@p-j-smith @paddyroddy
Add task to validate xnat_config.site_name (#84)
8e8d97c 
Fixes #76 

- check that `xnat_config.site_name` starts with a letter and contains
only letters, numbers, and underscores

---------

Co-authored-by: Patrick J. Roddy <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add task to validate xnat_config.site_name UCL-MIRSG/ansible-collection-infra
1 participant
@paddyroddy