Skip to content

Add a flag to events to hide them from the events overview#2108

Closed
aTrueYety wants to merge 7 commits into
masterfrom
erikhoff00/web-129-add-a-flag-to-events-to-hide-them-from-the-events-overview
Closed

Add a flag to events to hide them from the events overview#2108
aTrueYety wants to merge 7 commits into
masterfrom
erikhoff00/web-129-add-a-flag-to-events-to-hide-them-from-the-events-overview

Conversation

@aTrueYety

Copy link
Copy Markdown
Contributor

No description provided.

@aTrueYety aTrueYety self-assigned this Mar 10, 2026

if TYPE_CHECKING:
from rest_framework.request import Request

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

burde ikke trenge denne ifen siden type hinting eval blir forsinka av from __future__ import annotations

@hei98 hei98 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Koden ser bra ut og det fungerer:)

operations = [
migrations.AddField(
model_name='event',
name='is_hidden',

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.

Hvorfor kan ikke status feltet i Event brukes til dette formålet? Der kan man sette status til "privat".

Status sjekkes forsåvidt ikke i homepage, fikses i denne PRen.

Det mangler også et felt i frontend admin til å sette status

@robines robines changed the title Erikhoff00/web 129 add a flag to events to hide them from the events overview Add a flag to events to hide them from the events overview Mar 24, 2026
@aTrueYety

Copy link
Copy Markdown
Contributor Author

Restarted this in #2144 due to conflicting changes from other issue

@aTrueYety aTrueYety closed this Apr 7, 2026
@robines
robines deleted the erikhoff00/web-129-add-a-flag-to-events-to-hide-them-from-the-events-overview branch June 19, 2026 21:33
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.

4 participants