Would it be possible to remove the Notification module's checkbox for "Do not send notifications for this update" from the FAQ Ask a Question form? It only appears for authenticated users. See attached screenshot.

Comments

stenjo’s picture

Category: bug » feature
StatusFileSize
new8.41 KB

Is this not an issue for the Notification module?
I see issues if the FAQ-Ask module should be responsible for removing stuff non-core modules are adding to the question form.
On the other hand the question form should be clean and without unneccessary stuff.

I'm not sure this is a bug anyway. Feature request, yes. Bug? No, but I'm willling to listen to arguments for other points of view.

Checking the settings of the notification module at http://example.com/admin/config/notify
Notification module settings
The notification for faq content types can be turned off. Is this not sufficient? If not, it would be helpful with a scenario description making a case for shaping up the Notification module within the FAQ-Ask module.

One option is to ask for a more granulated permission setting for the notification module. Still feel that is the right place for a request like this. I see the Notify module is still in development for a D7 version. Maybe a request like this might be acceptable?

jlea9378’s picture

Feature request makes sense. I wasn't really sure which to consider it.

I'm using:
Messaging 7.x-1.0-alpha2+1-dev (2012-Mar-04)
Notifications 7.x-1.x-dev (2011-Dec-09)

I don't see the option in your screenshot, and I don't have anything at /admin/config/notify
I'm guessing that must be a different module?

What I'm requesting is to add a check to see if the Notifications module is installed, and if so, hide the "Do not send notifications for this update" field on the short Ask a Question form. It's fine on the full form. I just don't want to disable subscriptions/notifications all together for the entire content type.

nancydru’s picture

Has the Notifications module changed to where they support unpublished content? When I first wrote this, there were no notification-type modules that did.

stenjo’s picture

Yes, I was thinking of the Notify module. That one does not seem to have been touched since July last year + its buggy and not working well.
The Notifications module I have not checked. I'll see how that one works.

stenjo’s picture

How is the Notifications module set up so that logged in users get the checkbox you want removed?
It does not appear by default to me when I install the Messaging and the Notifications modules.

jlea9378’s picture

I uninstalled and reinstalled both modules and it is appearing still on the short form and the full form. It looks like this permission is what makes it appear, which is only enabled for Admins by default I think. Do you have it enabled for your role?

under "Notifications" heading
Skip notifications
description: Make changes with an option to skip notifications when available.

stenjo’s picture

Status: Active » Closed (works as designed)

I've been looking at this and I do now get the option box displayed on the question form for the admin user.
Still feels this is something more appropriate to fix within the notfication module itself. Especially when this is a settings issue there (should be possible to choose where and if you want to display such a box for all users)

I feel a little bad having to turn this down, but at the same time I think it is a good idea to focus on the core functionality on the module and not account for all the various optional modules that might affect faq_ask

stenjo’s picture

StatusFileSize
new24.97 KB

Added note:
When switching off the Subscription for the content type at admin/config/messaging/subscriptions/content:
Subscription settings
The checkbox is removed.
Does this solve the problem for you?
Are there any reasons for keeping them there?

jlea9378’s picture

In our situation, disabling subscriptions would be an acceptable solution since the majority of our site's visitors do not log in to the website and thus wouldn't be using the subscription feature. However, other FAQ Ask users who use the Notifications module may encounter this issue and may not like that solution. For that reason I created a feature request issue with the Notifications module: #1632962: Allow suppression of "Do Not Send Notification" checkbox

stenjo’s picture

Thank you. Then this thread has a reference to the issue that might fix it for others that might have the same problem.
Anyone feeling they have added scenarios; feel free to reopen this issue adding the extra information.