Skip to content

Update observation status asynchronusly with HTMX#1460

Merged
Fingel merged 7 commits intodevfrom
htmx-dan
Mar 19, 2026
Merged

Update observation status asynchronusly with HTMX#1460
Fingel merged 7 commits intodevfrom
htmx-dan

Conversation

@Fingel
Copy link
Contributor

@Fingel Fingel commented Mar 19, 2026

Makes updating the observation status reactive. Also splits the django messages html into into a partial so it can be re-used for other HTMX interactions.

@Fingel Fingel requested a review from davner March 19, 2026 18:22
@jchate6 jchate6 moved this to Needs Review in TOM Toolkit Mar 19, 2026
Fingel and others added 4 commits March 19, 2026 13:28
@Fingel Fingel changed the title Initial htmx swap Update observation status asynchronusly with HTMX Mar 19, 2026
Copy link
Collaborator

@davner davner left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

<h4>Observations</h4>
<a href="{% url 'targets:detail' pk=target.id %}?update_status=True" title="Update status of observations for target" class="btn btn-primary">Update Observations Status</a>
{% observation_list object %}
<!-- htmx loading -->
Copy link
Collaborator

Choose a reason for hiding this comment

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

Unnecessary comment.

@Fingel Fingel requested a review from phycodurus March 19, 2026 21:45
@Fingel Fingel merged commit fe0704f into dev Mar 19, 2026
26 checks passed
@github-project-automation github-project-automation bot moved this from Needs Review to Merged (to dev) in TOM Toolkit Mar 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Merged (to dev)

Development

Successfully merging this pull request may close these issues.

3 participants