Skip to content

API for account creation#13

Draft
cmyui wants to merge 6 commits intomasterfrom
account-creation
Draft

API for account creation#13
cmyui wants to merge 6 commits intomasterfrom
account-creation

Conversation

@cmyui
Copy link
Member

@cmyui cmyui commented Jun 14, 2025

No description provided.

client_ip_address: str,
client_user_agent: str,
) -> AuthorizationGrant | Error:
# "SELECT value_int FROM system_settings WHERE name = 'registrations_enabled'").Scan(&enabled)
Copy link
Member Author

Choose a reason for hiding this comment

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

TODO before merge?

Copy link
Member Author

Choose a reason for hiding this comment

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

Alternative move to amplitude FF?

},
)

# TODO: what is the "Y" cookie?
Copy link
Member Author

Choose a reason for hiding this comment

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

return Error(
error_code=ErrorCode.BAD_REQUEST,
user_feedback="Username does not meet requirements.",
)
Copy link
Member Author

@cmyui cmyui Jun 14, 2025

Choose a reason for hiding this comment

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

maybe backport pre-merge

"X-Ripple-Token",
value=response.unhashed_access_token,
expires=60 * 60 * 24 * 30,
domain="next.akatsuki.gg",
Copy link
Member Author

Choose a reason for hiding this comment

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

TODO conditional based on caller for migration

@cmyui cmyui marked this pull request as ready for review June 14, 2025 11:30
@cmyui cmyui requested a review from infernalfire72 as a code owner June 14, 2025 11:30
@cmyui cmyui marked this pull request as draft June 14, 2025 11:31
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.

1 participant