Skip to content

Joao/deck2 templates#535

Closed
JoaoVitorLobo wants to merge 0 commit intostagingfrom
joao/deck2-templates
Closed

Joao/deck2 templates#535
JoaoVitorLobo wants to merge 0 commit intostagingfrom
joao/deck2-templates

Conversation

@JoaoVitorLobo
Copy link
Copy Markdown

No description provided.

palestra no nosso evento. Ficaríamos muito gratos pela
oportunidade de recebê-l<b style="color: red">o/a/e</b> se
aceitar este convite.
Sou {{.MemberArticle}} {{.Member}}, um orgulhoso membro da
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

"um orgulhoso membro" also needs the article


const createSpeakerTemplateVariables = () => {
const endDate = new Date(eventStore.selectedEvent?.end || 0);
const memberGender = authStore.member?.contactObject?.gender ?? "OTHER";
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The idea is to force the user to fill all the information. The gender of the member and the speaker should always exist when getting to this step.

Comment thread frontend/src/lib/templates.ts Outdated
return { article: "o", suffix: "" };
case "FEMALE":
return { article: "a", suffix: "a" };
default:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

it misses the "OTHER" gender case

Copy link
Copy Markdown
Author

@JoaoVitorLobo JoaoVitorLobo Nov 24, 2025

Choose a reason for hiding this comment

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

wouldn't the default case and the "OTHER" case be the same?

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