Skip to content

feat(django messages): Auto close the django messages after a few seconds.#165

Merged
samoehlert merged 2 commits intomainfrom
topic/soehlert/disappearing_messages
Jul 9, 2025
Merged

feat(django messages): Auto close the django messages after a few seconds.#165
samoehlert merged 2 commits intomainfrom
topic/soehlert/disappearing_messages

Conversation

@samoehlert
Copy link
Copy Markdown
Collaborator

This should improve the UX

@samoehlert samoehlert marked this pull request as draft July 9, 2025 06:49
…s auto disappear after a few seconds. This is better UX than requiring the user to actually click the x to close the message
@samoehlert samoehlert force-pushed the topic/soehlert/disappearing_messages branch from 29eb503 to 68f7f66 Compare July 9, 2025 06:52
@samoehlert samoehlert added the ux User experience issues label Jul 9, 2025
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 9, 2025

File Coverage
All files 83%
config/consumers.py 78%
config/urls.py 69%
config/settings/base.py 69%
config/settings/local.py 73%
scram/route_manager/admin.py 85%
scram/route_manager/models.py 70%
scram/route_manager/views.py 88%
scram/route_manager/api/serializers.py 73%
scram/route_manager/api/views.py 83%
scram/shared/shared_code.py 56%
scram/templates/403.html 91%
scram/templates/404.html 91%
scram/templates/base.html 99%

Minimum allowed coverage is 50%

Generated by 🐒 cobertura-action against 13594a9

Copy link
Copy Markdown
Collaborator

@crankynetman crankynetman left a comment

Choose a reason for hiding this comment

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

This is awesome, approved.

@samoehlert samoehlert marked this pull request as ready for review July 9, 2025 16:39
@samoehlert samoehlert merged commit 0a78988 into main Jul 9, 2025
19 checks passed
@samoehlert samoehlert deleted the topic/soehlert/disappearing_messages branch July 9, 2025 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ux User experience issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants