Add user key field #1

Merged
calliope merged 20 commits from feature/add-user-key into master 2026-01-10 23:15:38 +00:00
Contributor
No description provided.
Kodomu left a comment
Owner

Some small suggestions.

Some small suggestions.
@ -40,0 +36,4 @@
$user = User::create([
'first_name' => $this->option('first_name') ?? '',
'last_name' => $this->option('last_name') ?? '',
'email' => $this->option('email') ?? '',
Owner

Should these fields be empty instead of null

Should these fields be empty instead of null
Author
Contributor

Shouldn't matter

Shouldn't matter
Kodomu marked this conversation as resolved
@ -39,1 +37,3 @@
'email' => 'required|email|max:255|unique:users',
'first_name' => 'string|max:255',
'last_name' => 'string|max:255',
'email' => 'email|max:255',
Owner

Again here, should we use nullable instead of just removing required?

Again here, should we use nullable instead of just removing required?
Kodomu marked this conversation as resolved
@ -0,0 +4,4 @@
<x-logo class="h-10" />
<h1 class="text-2xl text-center mt-6">{{ __('auth.sign_up_complete.title') }}</h1>
<div class="text-base text-center mt-6">{{ __('auth.sign_up_complete.user_key_info') }}</div>
<div class="text-lg font-semibold text-center mt-6">{{ $user_key }}</div>
Owner

Let's use this here to make it easier to copy and present it better.

<div class="text-xl font-semibold text-center mt-6 font-mono select-all bg-primary-700 p-4 rounded"> {{ $user_key }}</div>
<p class="text-sm text-center mt-2 text-gray-400">Click to select and copy</p>
Let's use this here to make it easier to copy and present it better. ``` <div class="text-xl font-semibold text-center mt-6 font-mono select-all bg-primary-700 p-4 rounded"> {{ $user_key }}</div> <p class="text-sm text-center mt-2 text-gray-400">Click to select and copy</p> ```
calliope marked this conversation as resolved
Owner

What happens if email notifications go to a user that doesn't have an email assigned?

What happens if email notifications go to a user that doesn't have an email assigned?
Make obvious that these details are optional
Owner

This is most likely ready now

This is most likely ready now
calliope merged commit 09f0f232a4 into master 2026-01-10 23:15:38 +00:00
calliope deleted branch feature/add-user-key 2026-01-10 23:15:38 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Ko-Net/Paymenter!1
No description provided.