Skip to content

Feature/comments on specific objects (some ideas) - #4959

Closed
frjo wants to merge 14 commits into
feature/comments-on-specific-objectsfrom
feature/comments-on-specific-objects-2
Closed

Feature/comments on specific objects (some ideas)#4959
frjo wants to merge 14 commits into
feature/comments-on-specific-objectsfrom
feature/comments-on-specific-objects-2

Conversation

@frjo

@frjo frjo commented Aug 21, 2026

Copy link
Copy Markdown
Member

No description provided.

@frjo
frjo force-pushed the feature/comments-on-specific-objects-2 branch 2 times, most recently from 3cf0143 to a318385 Compare August 21, 2026 07:59
@frjo
frjo changed the base branch from main to feature/comments-on-specific-objects August 21, 2026 08:00
@frjo
frjo force-pushed the feature/comments-on-specific-objects-2 branch from a318385 to 07c4dbc Compare August 21, 2026 08:05
@wes-otf

wes-otf commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

This is looking pretty good! @frjo when you get a chance can you add a desc/test steps? Looking through the code I think I get the main ideas though

<div>
<div class="px-8 border-t h-timeline">
{% for activity in activities %}
{# `|length` evaluates the queryset once and caches it, so the count below and #}

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.

nice optimization!

@frjo

frjo commented Aug 25, 2026

Copy link
Copy Markdown
Member Author

@wes-otf This is mostly LLM generated, see all the test that LLM love to add. I thought it could give some neat ideas. Parts might be overly complicated.

@frjo
frjo force-pushed the feature/comments-on-specific-objects branch from a740ea0 to 3040f45 Compare August 25, 2026 10:38
@frjo frjo closed this Aug 25, 2026
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