Closed (works as designed)
Project:
FAQ_Ask
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
21 May 2012 at 16:09 UTC
Updated:
13 Jun 2012 at 19:45 UTC
Jump to comment: Most recent file
Comments
Comment #1
stenjo commentedIs 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

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?
Comment #2
jlea9378 commentedFeature 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.
Comment #3
nancydruHas the Notifications module changed to where they support unpublished content? When I first wrote this, there were no notification-type modules that did.
Comment #4
stenjo commentedYes, 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.
Comment #5
stenjo commentedHow 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.
Comment #6
jlea9378 commentedI 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.
Comment #7
stenjo commentedI'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
Comment #8
stenjo commentedAdded note:

When switching off the Subscription for the content type at
admin/config/messaging/subscriptions/content:The checkbox is removed.
Does this solve the problem for you?
Are there any reasons for keeping them there?
Comment #9
jlea9378 commentedIn 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
Comment #10
stenjo commentedThank 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.