Skip to content

feat(events): the published panel says who the event is for - #136

Open
Saul-Gomez-J wants to merge 1 commit into
mainfrom
feature/event-access
Open

Saul-Gomez-J wants to merge 1 commit into
mainfrom
feature/event-access

Conversation

@Saul-Gomez-J

Copy link
Copy Markdown
Contributor

Why

Every published informative event showed "Abierto a todos", which is false for the workshops held as Discord events of the private NaN server.

What

  • Read ev.access (B-28, backend PR helmcode/nan-events-backend#3) in the published panel of /events/{slug}:
    • members → "Solo para la comunidad de NaN" / "NaN members only"
    • open → "Abierto a todos" / "Open to everyone"
    • empty → neutral "Evento anunciado" / "Event announced"
  • CTA ("Ir al evento") unchanged. The field is copy only; nothing is enforced.
  • EventInfo/PublicEventSummary carry access; calendar fixture updated.
  • Guards in eventsUiGuards.test.ts: the page branches on the field with the neutral fallback, both locales carry the three variants and only the open one says "everyone".

Safe to merge before the backend: until the API sends access, the page falls through to the neutral copy.

npm test (1327 passed) and astro check (0 errors) are green.

🤖 Generated with Claude Code

https://claude.ai/code/session_014ToscGMqPkLEBZ6x8MhcP7

"Abierto a todos" was shown on every published informative event, which
is false for the workshops held as Discord events of the private NaN
server. Read `ev.access` (B-28, helmcode/nan-events-backend#3) and pick
the copy: "members" → "Solo para la comunidad de NaN", "open" → "Abierto
a todos", empty → neutral "Evento anunciado". The CTA is unchanged.

Guards: the page branches on the field with the neutral fallback, both
locales carry the three variants and only the open one says "everyone".

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014ToscGMqPkLEBZ6x8MhcP7
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.

2 participants