Skip to content

docs: migrated the Reports component from the old docs to the new docs#238

Merged
adiati98 merged 24 commits intomautic:5.xfrom
OfficialJhimmy:feyijimi-new-wth
Mar 8, 2026
Merged

docs: migrated the Reports component from the old docs to the new docs#238
adiati98 merged 24 commits intomautic:5.xfrom
OfficialJhimmy:feyijimi-new-wth

Conversation

@OfficialJhimmy
Copy link
Copy Markdown

No description provided.

@OfficialJhimmy
Copy link
Copy Markdown
Author

OfficialJhimmy commented Apr 15, 2025

This pull request migrates over the documentation for the Reports Component in the reStructuredText (RST) format. It addresses #199.

@RCheesley RCheesley self-requested a review April 22, 2025 08:45
@favour-chibueze favour-chibueze added the writetechhub Pull requests from writetechhub's team label Apr 22, 2025
Copy link
Copy Markdown
Member

@RCheesley RCheesley left a comment

Choose a reason for hiding this comment

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

Thanks for making a start at this @OfficialJhimmy - there's a lot of Vale fixes to address and probably more lurking beneath your use of vale off.

Please remove all of those, and we'll tell you where to use it if it's absolutely necessary.

Shout if you need some help running Vale in your editor, it should have picked all of these up while you were typing!

Comment thread docs/components/reports.rst Outdated
Comment thread docs/components/reports.rst Outdated
Comment thread docs/components/reports.rst Outdated
Comment thread docs/components/reports.rst Outdated
Comment thread docs/components/reports.rst Outdated
Comment thread docs/components/reports.rst Outdated
Comment thread docs/components/reports.rst Outdated
Comment thread docs/components/reports.rst Outdated
Comment thread docs/components/reports.rst Outdated
Comment thread docs/components/reports.rst Outdated
@adiati98 adiati98 requested a review from a team as a code owner January 9, 2026 22:14
@adiati98 adiati98 requested review from adiati98 and favour-chibueze and removed request for a team January 9, 2026 22:14
Comment thread docs/components/reports.rst Outdated
Comment thread docs/components/reports.rst Outdated
Comment thread docs/components/reports.rst Outdated
@adiati98 adiati98 requested a review from escopecz March 4, 2026 14:43
@escopecz
Copy link
Copy Markdown
Member

escopecz commented Mar 4, 2026

There are conflicts

Copy link
Copy Markdown
Member

@escopecz escopecz left a comment

Choose a reason for hiding this comment

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

Looks fine. Just a few suggestions

Comment thread docs/links/mautic_chart_helpers_folder_52.py
Comment thread docs/plugin_extensions/reports.rst Outdated
Comment thread docs/plugin_extensions/reports.rst Outdated
Comment thread docs/plugin_extensions/reports.rst Outdated
Comment thread docs/plugin_extensions/reports.rst Outdated
Comment thread docs/plugin_extensions/reports.rst Outdated
adiati98 and others added 2 commits March 6, 2026 07:13
Co-authored-by: John Linhart <jan@linhart.email>
@adiati98 adiati98 requested a review from escopecz March 6, 2026 16:15
escopecz
escopecz previously approved these changes Mar 6, 2026
Copy link
Copy Markdown
Contributor

@adiati98 adiati98 left a comment

Choose a reason for hiding this comment

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

Thank you @OfficialJhimmy, @escopecz & @RCheesley! 🚀

@adiati98 adiati98 dismissed RCheesley’s stale review March 8, 2026 08:13

All suggestions have been addressed.

@adiati98 adiati98 merged commit 5d81b74 into mautic:5.x Mar 8, 2026
3 checks passed
@github-project-automation github-project-automation Bot moved this from 🕰️ In progress to 🥳 Done in Mautic's No/Low/Code Tasks Mar 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

first-contribution writetechhub Pull requests from writetechhub's team

Projects

Status: 🥳 Done

Development

Successfully merging this pull request may close these issues.

6 participants