Skip to content

Conversation

@itsisak
Copy link
Contributor

@itsisak itsisak commented Sep 11, 2025

Description

Removes the option for sending anonymous contact forms.

This could be an issue if this is being linked outside of abakus.no intended for externals to use to contact abakus as it is now not possible to submit the contact form without being logged in. The path abakus.no/contact seems like a typical entry point for externals to contact abakus on if they dont know of other ways.

If this contact form is only intented for abakus users anyways it should be okay.

Testing

  • I have thoroughly tested my changes.

See related backend pr: webkom/lego#3842

Resolves ABA-1496

@itsisak itsisak requested review from a team and magnusbrecke September 11, 2025 13:41
@itsisak itsisak self-assigned this Sep 11, 2025
@itsisak itsisak added question Pull requests with an unresolved question review-needed Pull requests that need review small-fix Pull requests that fix something small labels Sep 11, 2025
@linear
Copy link

linear bot commented Sep 11, 2025

Copy link
Contributor

@falbru falbru left a comment

Choose a reason for hiding this comment

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

Nice!

@itsisak
Copy link
Contributor Author

itsisak commented Sep 15, 2025

Page when not logged in:

Screenshot 2025-09-15 at 15 38 45

Copy link
Contributor

@Viljen789 Viljen789 left a comment

Choose a reason for hiding this comment

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

Looks good! Nice change

Copy link
Contributor

@ch0rizo ch0rizo left a comment

Choose a reason for hiding this comment

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

So freaky clean! 🪨

@itsisak itsisak force-pushed the remove-anonymous-contact-option branch from 3367bb4 to 5e0bd0c Compare October 11, 2025 11:45
@itsisak itsisak merged commit 0301fd3 into master Oct 11, 2025
7 checks passed
@itsisak itsisak deleted the remove-anonymous-contact-option branch October 11, 2025 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

question Pull requests with an unresolved question review-needed Pull requests that need review small-fix Pull requests that fix something small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants