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.

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

Make site_name valid #77

Merged
merged 1 commit into from
Feb 21, 2024
Merged

Make site_name valid #77

merged 1 commit into from
Feb 21, 2024

Conversation

paddyroddy
Copy link
Member

@paddyroddy paddyroddy commented Feb 21, 2024

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.

@paddyroddy paddyroddy added the problem Something isn't working label Feb 21, 2024
@paddyroddy paddyroddy self-assigned this Feb 21, 2024
@paddyroddy paddyroddy requested a review from a team February 21, 2024 11:56
@paddyroddy paddyroddy merged commit 163a8dd into main Feb 21, 2024
3 checks passed
@paddyroddy paddyroddy deleted the site-name branch February 21, 2024 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
problem Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants