When enableMailSummaries is false in the configuration file, prevent accessing the "E-Mail Summaries" page in the account settings
#6774
Labels
Comments
|
@luke- if the solution is only to throw a 404 error in the controller, I can send a PR for it. |
|
@marc-farre Yes, in this case we should hide the navigation item in the user & admin area and a 404 is sufficient for the controller. |
marc-farre
added a commit
that referenced
this issue
Dec 22, 2023
…e, prevent accessing the E-Mail Summaries page in the account settings
14 tasks
|
@marc-farre Thanks! |
github-merge-queue bot
pushed a commit
that referenced
this issue
Dec 22, 2023
…e, prevent accessing the E-Mail Summaries page in the account settings (#6775)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What steps will reproduce the problem?
Disable E-Mail Summaries: https://docs.humhub.org/docs/admin/config-options/#activity ->
'enableMailSummaries' => falseGo to https://my-humhub.com/activity/user
What is the expected result?
404 not found
What do you get instead?
The page is displayed
Additional info
Original discussion: https://community.humhub.com/content/perma?id=273925
The text was updated successfully, but these errors were encountered: