Skip to content

assets: overrides: Show comments in timeline expanded by default#778

Merged
zzacharo merged 2 commits into
CERNDocumentServer:masterfrom
sakshamarora1:fix/comment_expanded
May 5, 2026
Merged

assets: overrides: Show comments in timeline expanded by default#778
zzacharo merged 2 commits into
CERNDocumentServer:masterfrom
sakshamarora1:fix/comment_expanded

Conversation

@sakshamarora1
Copy link
Copy Markdown
Contributor

@sakshamarora1 sakshamarora1 commented Apr 22, 2026

Screenshot 2026-05-05 at 15 38 21

@palkerecsenyi
Copy link
Copy Markdown
Member

Fyi this introduces a slight UI issue caused by a bug in Invenio Requests: inveniosoftware/invenio-requests#605

@kpsherva
Copy link
Copy Markdown
Contributor

kpsherva commented May 4, 2026

Fyi this introduces a slight UI issue caused by a bug in Invenio Requests: inveniosoftware/invenio-requests#605

should this PR be closed, updated or the bug should be fixed in invenio-requests?

@sakshamarora1 sakshamarora1 force-pushed the fix/comment_expanded branch from 0a3ec8d to c2a5a11 Compare May 5, 2026 13:37
Copy link
Copy Markdown
Contributor

@zubeydecivelek zubeydecivelek left a comment

Choose a reason for hiding this comment

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

Looks good!

@zzacharo zzacharo merged commit 2b43916 into CERNDocumentServer:master May 5, 2026
3 checks passed
@github-project-automation github-project-automation Bot moved this from In review 🔍 to To release 🤖 in Sprint Q2 2026 ☀️ May 5, 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.

5 participants