Skip to content

Tickets#606

Draft
ThomasBow wants to merge 85 commits into
nextfrom
tickets
Draft

Tickets#606
ThomasBow wants to merge 85 commits into
nextfrom
tickets

Conversation

@ThomasBow
Copy link
Copy Markdown
Contributor

@ThomasBow ThomasBow commented Jan 7, 2026

Prerequisite / depends-on: #620
Prerequisite / depends-on: #632

@krestenlaust
Copy link
Copy Markdown
Member

@ThomasBow fyi, I just reformatted the code

Using the command from: https://github.com/f-klubben/stregsystemet/blob/next/.github/workflows/black.yml (without --check)

@krestenlaust
Copy link
Copy Markdown
Member

@ThomasBow can you note down what Initiv chair wanted?

@krestenlaust krestenlaust marked this pull request as draft April 4, 2026 23:23
@krestenlaust
Copy link
Copy Markdown
Member

Mby you should reintroduce #648 xd

@krestenlaust krestenlaust mentioned this pull request May 19, 2026
Copy link
Copy Markdown
Member

@krestenlaust krestenlaust left a comment

Choose a reason for hiding this comment

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

Image Can I has separate app for tickets? 👉 👈

Also we need to fix the refund PR first, then this is ready

@ThomasBow
Copy link
Copy Markdown
Contributor Author

True, of course you can seperate it

@ThomasBow
Copy link
Copy Markdown
Contributor Author

Or i can do it if you are too busy

@krestenlaust
Copy link
Copy Markdown
Member

Brogramming session? I don't mind doing the initial work (and maybe the rest as well)

@ThomasBow
Copy link
Copy Markdown
Contributor Author

Let's do it ❤️

Had to comment out code in stregsystem.models which require a circular
dependency
@krestenlaust
Copy link
Copy Markdown
Member

krestenlaust commented May 23, 2026

I've moved all models and tests, among other things, but there's some pesky logic baked into the Sale model, which should be externalized somehow.

And we'll need to figure out a good way to dynamically add extra pages to the member page to move the remaining logic and templating

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