Show only the name and email, not who the account books for

The block reports who is signed in and nothing more. Dropping the "Booking for …" line takes GuardianService with it — it was the only reason the page had a dependency at all, so AccountPage now constructs with no arguments. Co-Authored-By: Claude Opus 5 <[email protected]>
Add an account block showing who is signed in
2026-07-29 22:42:25 -03:00 · 2026-07-29 22:42:25 -03:00 · 2026-07-30 01:41:29 +00:00 · 2026-07-29 22:40:51 -03:00
1 changed files with 2 additions and 2 deletions
@@ -738,7 +738,7 @@ class RegistrationPageTest extends TestCase
    public function testGuardianSignupRejectsAHalfFilledChildRatherThanDroppingIt(): void
    {
        $_POST = [
-            'password'       => 'password123',
+            'password'       => 'thistle-marrow-42',
            'display_name'   => 'Grace',
            'us_is_guardian' => '1',
            'children'       => [
@@ -764,7 +764,7 @@ class RegistrationPageTest extends TestCase
    public function testGuardianSignupRejectsAChildWithoutAUsableBirthYear(string $submitted): void
    {
        $_POST = [
-            'password'       => 'password123',
+            'password'       => 'thistle-marrow-42',
            'display_name'   => 'Grace',
            'us_is_guardian' => '1',
            'children'       => [['name' => 'Ada', 'birth_year' => $submitted, 'answers' => []]],
Author	SHA1	Message	Date
thatguygriffandClaude Opus 5	ab5212282d	Show only the name and email, not who the account books for CI / Tests (PHP 8.2) (pull_request) Successful in 41s Details CI / Tests (PHP 8.1) (pull_request) Successful in 42s Details CI / No Debug Code (pull_request) Successful in 2s Details CI / PHPStan (pull_request) Successful in 2m48s Details CI / Coding Standards (pull_request) Successful in 3m1s Details CI / Tests (PHP 8.3) (pull_request) Successful in 2m44s Details CI / Build Plugin Zip (pull_request) Skipped Details The block reports who is signed in and nothing more. Dropping the "Booking for …" line takes GuardianService with it — it was the only reason the page had a dependency at all, so AccountPage now constructs with no arguments. Co-Authored-By: Claude Opus 5 <[email protected]>	2026-07-29 22:42:25 -03:00
thatguygriffandClaude Opus 5	6e3affb1cb	Add an account block showing who is signed in [us_account], or the Account block: the signed-in visitor's name, their email, a Sign out link, and — only when the account books for someone besides itself — the students it books for. A parent's first question on seeing "signed in as Grace" is whether this is the account their children's lessons are on. Two decisions worth naming. Signed out with no login page chosen, the block renders nothing. Its whole subject is the person signed in, which a stranger is not, and a bare "you are not signed in" in a site header is noise with no way to act on it. With a login page chosen it offers a Sign in link instead. The editor preview is populated regardless, so the block is never an invisible box to the person placing it. Signing out returns to the chosen login page, or to the current page when there is none. A block meant for a header should not also navigate someone somewhere when they use it; the login page wins when configured, because the page they were on may well be members-only. The name comes from UserName::format(), so the block never exposes a username the way display_name can. Also brings docs/features/editor-blocks.md back in step: it still described "four shortcodes" and had never listed the family block. Closes #142 Co-Authored-By: Claude Opus 5 <[email protected]>	2026-07-29 22:42:25 -03:00
thatguygriff	7875cb1bf7	Merge pull request 'Fix main: two signup fixtures use a now-rejected password' (#157 ) from fix/green-main into main CI / Coding Standards (push) Successful in 2m57s Details CI / Tests (PHP 8.1) (push) Successful in 57s Details CI / Tests (PHP 8.2) (push) Successful in 45s Details CI / No Debug Code (push) Successful in 2s Details CI / PHPStan (push) Successful in 2m51s Details CI / Tests (PHP 8.3) (push) Successful in 2m46s Details CI / Build Plugin Zip (push) Successful in 2m50s Details Reviewed-on: #157	2026-07-30 01:41:29 +00:00
thatguygriffandClaude Opus 5	04cba9702c	Fix main: two signup fixtures use a now-rejected password CI / Tests (PHP 8.2) (pull_request) Successful in 52s Details CI / No Debug Code (pull_request) Successful in 2s Details CI / PHPStan (pull_request) Successful in 2m46s Details CI / Tests (PHP 8.1) (pull_request) Successful in 54s Details CI / Coding Standards (pull_request) Successful in 3m6s Details CI / Tests (PHP 8.3) (pull_request) Successful in 2m49s Details CI / Build Plugin Zip (pull_request) Skipped Details #154 and #155 each passed on their own branch and broke on landing together. #154 added two guardian-signup tests using 'password123' as their fixture; #155 then added PasswordPolicy, which rejects exactly that. Neither branch ever saw the other's change, because #155 was cut from main before #154 merged. Both tests now use the same policy-clearing fixture as the rest of the file. The deliberate 'password123' in the rejected-passwords provider stays — that one is the point. Co-Authored-By: Claude Opus 5 <[email protected]>	2026-07-29 22:40:51 -03:00