Skip to content
This repository has been archived by the owner on Feb 11, 2025. It is now read-only.

Add HTMX-aware messages-middleware #107

Merged
merged 2 commits into from
Oct 25, 2024
Merged

Add HTMX-aware messages-middleware #107

merged 2 commits into from
Oct 25, 2024

Conversation

hmpf
Copy link
Collaborator

@hmpf hmpf commented Sep 3, 2024 •
edited
Loading

Copy (django.contrib) messages list to response header so HTMx can easily access them.

@hmpf hmpf self-assigned this Sep 3, 2024
@hmpf hmpf requested review from a team September 3, 2024 11:52
@hmpf hmpf added the after-demo Need not work for demo label Sep 3, 2024
@hmpf hmpf removed the after-demo Need not work for demo label Sep 13, 2024
@podliashanyk podliashanyk mentioned this pull request Oct 16, 2024
Open
@podliashanyk podliashanyk mentioned this pull request Oct 24, 2024
Open
lunkwill42
lunkwill42 approved these changes Oct 24, 2024
View reviewed changes
Copy link
Member

@lunkwill42 lunkwill42 left a comment

Choose a reason for hiding this comment

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

I just see a lot more legacy code here, but that seems to be all the rage in argus-htmx ;-)

@hmpf hmpf merged commit a6a3202 into Uninett:main Oct 25, 2024
1 of 2 checks passed
@hmpf hmpf deleted the messages-middleware branch October 25, 2024 08:19
johannaengland added a commit that referenced this pull request Nov 4, 2024
@johannaengland
Reformat middleware file using ruff
bc456a5 
This was forgotten in #107
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@lunkwill42 lunkwill42 lunkwill42 approved these changes

Assignees

@hmpf hmpf

Labels
None yet
Projects
HTMXify Argus
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hmpf @lunkwill42