Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update mautic with composer #243

Merged
merged 10 commits into from
Feb 25, 2024
Merged

Update mautic with composer #243

merged 10 commits into from
Feb 25, 2024

Conversation

jagtapreshma
Copy link
Contributor

No description provided.

@RCheesley
Copy link
Member

@all-contributors please add @jagtapreshma for docs

Copy link
Contributor

@RCheesley

I've put up a pull request to add @jagtapreshma! 🎉

Copy link
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 the PR @jagtapreshma - great work and welcome to the community! 🚀

I've spotted a few small issues which we hadn't picked up before as this file hasn't been edited since we added some stricter checks.

Hopefully it all makes sense, but please let me know if anything needs explanation!

docs/getting_started/how_to_update_mautic.rst Outdated Show resolved Hide resolved
docs/getting_started/how_to_update_mautic.rst Outdated Show resolved Hide resolved
docs/getting_started/how_to_update_mautic.rst Outdated Show resolved Hide resolved
docs/getting_started/how_to_update_mautic.rst Outdated Show resolved Hide resolved
docs/getting_started/how_to_update_mautic.rst Outdated Show resolved Hide resolved
docs/getting_started/how_to_update_mautic.rst Outdated Show resolved Hide resolved
docs/getting_started/how_to_update_mautic.rst Outdated Show resolved Hide resolved
docs/getting_started/how_to_update_mautic.rst Outdated Show resolved Hide resolved
docs/getting_started/how_to_update_mautic.rst Outdated Show resolved Hide resolved
docs/getting_started/how_to_update_mautic.rst Outdated Show resolved Hide resolved
Copy link
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.

Just the title underline then we're good to go!

docs/getting_started/how_to_update_mautic.rst Outdated Show resolved Hide resolved
docs/getting_started/how_to_update_mautic.rst Outdated Show resolved Hide resolved
docs/getting_started/how_to_update_mautic.rst Outdated Show resolved Hide resolved
docs/getting_started/how_to_update_mautic.rst Outdated Show resolved Hide resolved
docs/getting_started/how_to_update_mautic.rst Outdated Show resolved Hide resolved
@RCheesley
Copy link
Member

Good to go, the failing build is due to some broken links which @fakela is going to fix in a separate PR.

@RCheesley RCheesley merged commit bfdd86e into mautic:5.x Feb 25, 2024
1 of 2 checks passed
@fakela
Copy link
Member

fakela commented Feb 25, 2024

I'll work on fixing the failing links this week

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.

3 participants