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

Possibility for more control over dismissability of banner #32

Open
luzpaz opened this issue Mar 13, 2018 · 14 comments
Open

Possibility for more control over dismissability of banner #32

luzpaz opened this issue Mar 13, 2018 · 14 comments

Comments

@luzpaz
Copy link

luzpaz commented Mar 13, 2018

Currently at the top of the FreeCAD mantisbt we use the announce plugin (as an intrusive banner) to educate users how to open tickets. It's cut down immensely on crap tickets that are hastily opened and hopelessly incomplete. The problem is that users can dismiss this banner and then forget in the future to follow the directions. If we make the banner indissmissable, then all the power users are subject to this enormous banner taking up screen real-estate.

The ask: Would it be possible to create a rule that makes the banner indismissable to reporters, and dismissable to everyone with a higher permission ?

@dregad
Copy link
Member

dregad commented Mar 21, 2018

It would certainly be feasible to transform the "dismissable" flag to an access level. It would require a schema change and some UI adjustments.

That being said, based on your use-case description, I'm wondering whether the Announce plugin is really the right tool for the job though. It seems to me that a more targeted message, appearing only on the MantisBT bug report page, would be more appropriate.

@luzpaz
Copy link
Author

luzpaz commented Mar 21, 2018

That being said, based on your use-case description, I'm wondering whether the Announce plugin is really the right tool for the job though. It seems to me that a more targeted message, appearing only on the MantisBT bug report page, would be more appropriate.

Perhaps, but psychologically I want to prepare the user to know what it means to make a bug report. I don't want to, last minute, for the user to be surprised by what hoops we're asking them to jump through. That is why I'd rather have 'reporters' see it on every page of the bugtracker instead of just the bug reporting page

@vboctor
Copy link
Member

vboctor commented Mar 21, 2018

@luzpaz I was thinking about the same feature, but the solution I had in mind was more inline with how Github solves this. Basically, to do be able to configure default values for description, additional info, and steps to reproduce. Such configs can be set globally via config/config_inc.php or on a per project basis. Such templates would apply to all access levels.

@atrol
Copy link
Member

atrol commented Mar 21, 2018

@vboctor for what you want, the Snippets plugin could be extended (e.g. additional propertyautofill_field for snippets + move global snippet settings to individual snippets)

@vboctor
Copy link
Member

vboctor commented Mar 21, 2018

@atrol I think this is better handled in core, since it not a snippet and it would be consistent with ability for core to default other native fields.

@luzpaz
Copy link
Author

luzpaz commented Mar 23, 2018

@luzpaz I was thinking about the same feature, but the solution I had in mind was more inline with how Github solves this. Basically, to do be able to configure default values for description, additional info, and steps to reproduce. Such configs can be set globally via config/config_inc.php or on a per project basis. Such templates would apply to all access levels.

I think the config/config_inc.php method also makes sense. It would add another nice customization option for projects that would like to use this type of functionality.

@vboctor
Copy link
Member

vboctor commented Mar 24, 2018

Turns out that steps to reproduce and additional info already supported defaulting. I have added support for defaulting description via the following PR:

mantisbt/mantisbt#1320

@luzpaz
Copy link
Author

luzpaz commented Mar 27, 2018

Awesome. I look forward to testing this when it's merged. Any idea on the ETA ?

@luzpaz
Copy link
Author

luzpaz commented Mar 30, 2018

Great, it's been merged. Thank you. Just need to wait till the next mantis release to test it.

@luzpaz

This comment was marked as off-topic.

@dregad

This comment was marked as off-topic.

@luzpaz

This comment was marked as off-topic.

@dregad

This comment was marked as off-topic.

@luzpaz

This comment was marked as off-topic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants