Skip to content

Add autocomplete attribute to email form field - #36

Merged
janopae merged 2 commits into
mainfrom
feat-email-autocomplete
Oct 6, 2026
Merged

janopae merged 2 commits into
mainfrom
feat-email-autocomplete

Conversation

@polarbirke

Copy link
Copy Markdown
Member

No description provided.

@polarbirke
polarbirke requested a review from janopae October 2, 2026 17:21
@janopae

janopae commented Oct 5, 2026

Copy link
Copy Markdown
Member

Good catch! I'm going to merge this and adjust gba-innovationsfonds/src/Resources/templates/bundles/WebfactoryNewsletterRegistrationBundle/StartRegistration/form-partial.html.twig:27

Correct me, if I'm wrong: This does not cause any behavioural change in the Innovationsfonds downstream project, but only gets rid of the awkward {{ form_row(registrationForm.emailAddress, {'label': 'E-Mail-Adresse'}) }} we have to do (no change for categories, as this downstream uses none).

@polarbirke

@polarbirke

Copy link
Copy Markdown
Member Author

Dein AI-Kommentar hat nichts mit diesem PR zu tun.

@sebastiankugler

sebastiankugler commented Oct 5, 2026 •

Copy link
Copy Markdown
Member

Mir scheint, der Kommentar gehört zu #37 und ist fälschlicherweise hier gelandet. Ich glaube aber auch, es ist nicht gut, wenn @janopae's Claude per Kommentar Fragen an @polarbirke stellt.

@janopae

janopae commented Oct 6, 2026 •

Copy link
Copy Markdown
Member

Ich poste keine AI-Kommentare.

Der Kommentar gehört zu #37

@polarbirke

Copy link
Copy Markdown
Member Author

Sorry, du bist Opfer meiner Überempfindlichkeit geworden. Mein innerer Detektor sprang vor allem bei "Good catch!" und "… (no change for categories, as this downstream uses none)." an.

Humane Kommentare sind immer willkommen 😉

@janopae

janopae commented Oct 6, 2026

Copy link
Copy Markdown
Member

Ja, ist auch totale AI-Red-Flag :D Bin glaub ich auch wg. Open Source und Englisch in so ein corporate-Speak-Modus verfallen ;) Wahrscheinlich haben Claude und ich für so was einfach die gleichen Trainingsdaten gesehen ;)

Aber Claude macht AFAIK keine britische Rechtschreibung (die ich mir irgendwann mal wegen JFE und YP angewöhnt habe) ;)

behavioural

@janopae janopae left a comment

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.

Tippitoppi

@janopae
janopae merged commit 01f8ef8 into main Oct 6, 2026
3 checks passed
@janopae
janopae deleted the feat-email-autocomplete branch October 6, 2026 12:32
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.

3 participants