Ticket #2021 (closed enhancement: fixed)
Opened 6 years ago
Last modified 3 years ago
Add "New Ticket" message that can be used to warn people it is not a test or demo site.
| Reported by: | anonymous | Owned by: | cmlenz |
|---|---|---|---|
| Priority: | highest | Milestone: | 0.9 |
| Component: | ticket system | Version: | devel |
| Severity: | normal | Keywords: | |
| Cc: | Dowl | ||
| Release Notes: | |||
| API Changes: | |||
Description
A paramter in the ini should be created to provide a message on the Create New Ticket page.
On the projects.edgewall.com/trac site, this message should be a warning that it is not a test or demo site. The warning should include a link to a demo site.
I think people must stumble on the Create New Ticket and don't know better. If people had an opportunity to go to a demo site, I think most would.
Attachments
Change History
comment:1 Changed 6 years ago by anonymous
comment:2 Changed 6 years ago by anonymous
It aims to solve the same problem as #1961, although in a more configurable manner.
comment:3 Changed 6 years ago by cboos
- Owner changed from jonas to cboos
- Priority changed from normal to highest
- Status changed from new to assigned
- Version changed from 0.8.4 to devel
I think that's a good idea.
What about something like:
[ticket] new_ticket_policy = "Please check that there isn't already a bug or feature request similar to the one you're about to create. etc."
comment:4 Changed 6 years ago by eblot
There are three (IMHO too many) sources for strings displayed to users:
- ClearSilver templates
- Python code
- trac.ini
I would prefer the "template" way: it would give a better leverage to customize the new ticket page.
We already have site_header.cs and site_footer.cs, why not a site_newticket.cs ?
comment:5 Changed 6 years ago by cboos
Ah, didn't think about that.
Yes, a site_newticket.cs seems to be even a better idea.
(the .ini file is really not appropriate for putting
multi-line content, that was a "pis aller" :) )
Changed 6 years ago by cboos
- Attachment 2021-fix.patch added
Adds a new default template (site_newticket_policy.cs) and refactored the code to make it easier to add new default templates in the future.
comment:6 Changed 6 years ago by cboos
- Milestone set to 0.9
The proposed patch would be useful to have on p.e.c,
along with a customized site_newticket_policy.cs informing
people that This is not a test system ...
comment:7 Changed 6 years ago by cmlenz
- Owner changed from cboos to cmlenz
- Status changed from assigned to new
I don't like requiring an upgrade just to put site-specific template in place. Personally, I remove all the site_*.cs files that I don't use.
I've got a similar change in the queue that doesn't have this problem, and will check it in.
comment:8 Changed 6 years ago by cmlenz
- Status changed from new to assigned
comment:9 Changed 6 years ago by cmlenz
- Resolution set to fixed
- Status changed from assigned to closed
Implemented in [2272].
comment:10 Changed 3 years ago by Semion Dowl
- Cc Dowl added
- Keywords HerboMart : The Ultimate Destination For All Health Needs added
- Summary changed from Add "New Ticket" message that can be used to warn people it is not a test or demo site. to Add "New Ticket" message that can be used to warn people it is not a test or demo site.
- Type changed from enhancement to defect
HerboMart? : The Ultimate Destination For All Health Needs
comment:11 Changed 3 years ago by ebray
- Keywords HerboMart : The Ultimate Destination For All Health Needs removed
- Summary changed from Add "New Ticket" message that can be used to warn people it is not a test or demo site. to Add "New Ticket" message that can be used to warn people it is not a test or demo site.
- Type changed from defect to enhancement



I think this is a duplicate of #1961.