Skip to content

FE: Messages: Implement messages export - #740

Open
alexanderlz wants to merge 32 commits into
kafbat:mainfrom
alexanderlz:issues/688
Open

FE: Messages: Implement messages export#740
alexanderlz wants to merge 32 commits into
kafbat:mainfrom
alexanderlz:issues/688

Conversation

@alexanderlz

Copy link
Copy Markdown
  • Breaking change? (if so, please describe the impact and migration path for existing application instances)

What changes did you make? (Give an overview)

Added an option to download all messages that are displayed on page(as csv or json)
as per #688

useDataSaver method is reused here.

screenshot

Is there anything you'd like reviewers to focus on?

How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)

  • No need to
  • Manually (please, describe, if necessary)
  • Unit checks
  • Integration checks
  • Covered by existing automation

Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (e.g. ENVIRONMENT VARIABLES)
  • My changes generate no new warnings (e.g. Sonar is happy)
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged

Check out Contributing and Code of Conduct

A picture of a cute animal (not mandatory but encouraged)

@alexanderlz
alexanderlz requested a review from a team as a code owner December 30, 2024 19:53
@kapybro kapybro Bot added status/triage Issues pending maintainers triage status/triage/manual Manual triage in progress status/triage/completed Automatic triage completed and removed status/triage Issues pending maintainers triage labels Dec 30, 2024

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi alexanderlz! 👋

Welcome, and thank you for opening your first PR in the repo!

Please wait for triaging by our maintainers.

Please take a look at our contributing guide.

@Haarolean Haarolean left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi, thanks for the PR! Could you please take a look at the linter errors? And could we get some tests for the new functionality?

@Haarolean Haarolean changed the title Closes #688 FE: Messages: Implement messages export Jan 1, 2025
@Haarolean Haarolean added type/enhancement En enhancement/improvement to an already existing feature scope/frontend Related to frontend changes area/messages and removed status/triage/manual Manual triage in progress labels Jan 1, 2025
@alexanderlz

Copy link
Copy Markdown
Author

definitely (both of your questions).

Happy New Year!

@Haarolean

Copy link
Copy Markdown
Member

definitely (both of your questions).

Happy New Year!

likewise, happy new year 😊

Comment thread frontend/src/components/Topics/Topic/Messages/MessagesTable.tsx Outdated
Comment thread frontend/src/components/Topics/Topic/Messages/MessagesTable.tsx Outdated
@germanosin germanosin added the status/pending-design Pending UI design solution label Jan 6, 2025
@alexanderlz
alexanderlz requested a review from Haarolean January 7, 2025 11:41
@Haarolean Haarolean added the hacktoberfest-accepted PRs accepted towards hacktoberfest goal and will be counted as approved label Oct 5, 2025
@Haarolean Haarolean modified the milestones: 1.4, 1.5 Nov 10, 2025
@Haarolean Haarolean removed this from Release 1.4 Nov 10, 2025
@Haarolean
Haarolean self-requested a review November 10, 2025 11:24
@Haarolean Haarolean moved this to Todo in Release 1.5 Nov 10, 2025
@yeikel

yeikel commented Nov 16, 2025

Copy link
Copy Markdown
Collaborator

Hi team

What is the latest status on this one? What's blocking merging it?

Thanks!

@yeikel yeikel mentioned this pull request Nov 16, 2025
13 tasks
Comment thread frontend/src/components/Topics/Topic/Messages/Filters/Filters.tsx
@ibalat

This comment was marked as off-topic.

@georgea298

This comment was marked as off-topic.

@alexanderlz

Copy link
Copy Markdown
Author

@yeikel ^ up

@yeikel

yeikel commented Mar 23, 2026

Copy link
Copy Markdown
Collaborator

@yeikel ^ up

Not me but @Haarolean and team. I am looking forward for this one as well

@Haarolean

Copy link
Copy Markdown
Member

@alexanderlz can you rebase this please? let's merge

@Haarolean Haarolean moved this from Todo to PR Approved in Release 1.5 Mar 26, 2026
@yeikel yeikel mentioned this pull request Mar 29, 2026
2 tasks
@Haarolean Haarolean modified the milestones: 1.5, 1.6 Apr 17, 2026
@Haarolean Haarolean removed this from Release 1.5 Apr 17, 2026
@Haarolean Haarolean moved this to Todo in Release 1.6 Apr 29, 2026
@github-project-automation github-project-automation Bot moved this from Todo to PR Approved in Release 1.6 Apr 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/messages hacktoberfest-accepted PRs accepted towards hacktoberfest goal and will be counted as approved scope/frontend Related to frontend changes status/triage/completed Automatic triage completed type/enhancement En enhancement/improvement to an already existing feature

Projects

Status: PR Approved
Status: Dev Implementation

Development

Successfully merging this pull request may close these issues.

8 participants