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 gmail.rst #311

Merged
merged 3 commits into from
Nov 3, 2024
Merged

Update gmail.rst #311

merged 3 commits into from
Nov 3, 2024

Conversation

DJUserkent
Copy link
Contributor

the plugin no longer works - I've added text to say that.

the plugin no longer works
@RCheesley
Copy link
Member

Thanks for the PR @DJUserkent - do you think it might be better just to remove this page all together?

@RCheesley RCheesley requested review from a team, fakela and favour-chibueze and removed request for a team June 17, 2024 13:39
@DJUserkent
Copy link
Contributor Author

DJUserkent commented Jun 18, 2024 via email

@RCheesley
Copy link
Member

I'll leave this up to the @mautic/education-team-leaders to decide, I feel like if it's something which is broken then it shouldn't be in our docs until it's in a usable state. As far as I know this hasn't worked for at least 5+ years, we've tried reaching out to the developer who built it to ask if they could update it to no avail. The GitHub repo is still there for people to find, however.

@DJUserkent
Copy link
Contributor Author

DJUserkent commented Jun 19, 2024 via email

@fakela
Copy link
Member

fakela commented Jun 19, 2024

Thanks for the PR @DJUserkent - do you think it might be better just to remove this page all together?

We can remove this page and probably have a page for list of plugins that no longer works in case anyone wants to verify from the docs

@RCheesley
Copy link
Member

I think we should remove the page entirely. The docs will still be there in the 4.x version for reference in case someone needs to get them.

@RCheesley RCheesley merged commit f948eb2 into mautic:5.x Nov 3, 2024
2 checks passed
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.

4 participants