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

Remove Heroku Reference from Debug Command #3292

Merged
merged 3 commits into from
Jul 15, 2023

Conversation

StephenDaDev
Copy link
Member

Very few users are still using Heroku for hosting their instance of Modmail. As such, this comment could cause confusion in regard to where to find their logs. So, changing it to a general term, such as console, remedies this issue, and doesn't specifically target any one hosting method.

Copy link
Collaborator

@Taaku18 Taaku18 left a comment

Choose a reason for hiding this comment

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

Hi, you might've missed another reference lower down on L485. Other than that, looks good.

@Taaku18 Taaku18 added staged Staged for next version and removed in progress labels Jul 14, 2023
@Taaku18 Taaku18 merged commit d891ad1 into modmail-dev:development Jul 15, 2023
raidensakura pushed a commit to raidensakura/modmail that referenced this pull request Apr 10, 2024
* Very few users are still using Heroku for hosting, so this comment could cause confusion.

* Fix mistake

---------

Co-authored-by: Taku <[email protected]>
khakers pushed a commit to khakers/OpenModmail that referenced this pull request Jun 18, 2024
* Very few users are still using Heroku for hosting, so this comment could cause confusion.

* Fix mistake

---------

Co-authored-by: Taku <[email protected]>
(cherry picked from commit d891ad1)
khakers pushed a commit to khakers/OpenModmail that referenced this pull request Jun 18, 2024
* Very few users are still using Heroku for hosting, so this comment could cause confusion.

* Fix mistake

---------

Co-authored-by: Taku <[email protected]>
(cherry picked from commit d891ad1)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
staged Staged for next version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants