Don't show mail-notification settings when ENABLE_NOTIFY_MAIL is false #9129

Closed
opened 2025-11-02 08:30:23 -06:00 by GiteaMirror · 1 comment
Owner

Originally created by @uriesk on GitHub (Jun 25, 2022).

Description

I wasted time playing with mail-notification settings in the profile and adding / removing mail addresses.
Till i realized that there is a ENABLE_NOTIFY_MAIL setting, which is false per default.

I think the mail notification setting in the profile should not be available (grayed out), or there should be some
"Mail notifications are disabled by the Administration" or any other hint making it obvious that it is not available.

Screenshots

Screenshot_20220625_111716

Gitea Version

1.17.0-rc1

Can you reproduce the bug on the Gitea demo site?

Yes

Operating System

No response

Browser Version

Brave 1.40.105 Chromium: 103.0.5060.53

Originally created by @uriesk on GitHub (Jun 25, 2022). ### Description I wasted time playing with mail-notification settings in the profile and adding / removing mail addresses. Till i realized that there is a ENABLE_NOTIFY_MAIL setting, which is false per default. I think the mail notification setting in the profile should not be available (grayed out), or there should be some `"Mail notifications are disabled by the Administration"` or any other hint making it obvious that it is not available. ### Screenshots <img width="871" alt="Screenshot_20220625_111716" src="https://user-images.githubusercontent.com/13338250/175767852-c3cc1619-077d-4733-a2ea-2b64b646d3b0.png"> ### Gitea Version 1.17.0-rc1 ### Can you reproduce the bug on the Gitea demo site? Yes ### Operating System _No response_ ### Browser Version Brave 1.40.105 Chromium: 103.0.5060.53
GiteaMirror added the topic/uitype/bug labels 2025-11-02 08:30:23 -06:00
Author
Owner

@Francewhoa commented on GitHub (Dec 7, 2022):

This is another example of a user struggling with notification at https://discourse.gitea.io/u/JVC-MSF

I believe that @uriesk's suggestions above would resolve this challenge

@Francewhoa commented on GitHub (Dec 7, 2022): This is another example of a user struggling with notification at https://discourse.gitea.io/u/JVC-MSF I believe that @uriesk's suggestions above would resolve this challenge
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#9129