Skip to content

Conversation

IDzyre
Copy link
Contributor

@IDzyre IDzyre commented Feb 8, 2025

No description provided.

@bmcutler
Copy link
Member

@IDzyre
Any further updates to this documentation before we can review & merge?

@IDzyre
Copy link
Contributor Author

IDzyre commented Jul 28, 2025

The JSON is

"user_create_account": true/false,
"user_id_requirements": {
    "all": true,
    "require_name": false,
    "min_length": 6,
    "max_length": 25,
    "name_requirements": {
      "given_first": false,
      "given_name": 2,
      "family_name": 4
    },
    "require_email": false,
    "email_requirements": {
      "whole_email": false,
      "whole_prefix": false,
      "prefix_count": 6
    },
    "accepted_emails": {
    "gmail.com": true,
   }
}

Copy link
Member

@bmcutler bmcutler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've made some edits, and I think its ok to merge now.

@IDzyre, please review and make any additional revisions. I'd like to merge soon :)

@github-project-automation github-project-automation bot moved this from Seeking Reviewer to Awaiting Maintainer Review in Submitty Development Jul 28, 2025
@bmcutler bmcutler marked this pull request as ready for review July 28, 2025 21:08
@IDzyre IDzyre moved this from Awaiting Maintainer Review to Ready to Merge in Submitty Development Jul 28, 2025
@IDzyre
Copy link
Contributor Author

IDzyre commented Jul 28, 2025

Done.

@bmcutler bmcutler merged commit f35395d into main Jul 28, 2025
3 checks passed
@bmcutler bmcutler deleted the self-account-creation branch July 28, 2025 21:42
@github-project-automation github-project-automation bot moved this from Ready to Merge to Done in Submitty Development Jul 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants