Modify ↓
#11036 closed enhancement (fixed)
Use Option class for defining options
Reported by: | Ryan J Ollos | Owned by: | Graham Miln |
---|---|---|---|
Priority: | normal | Component: | MilnCheckboxPlugin |
Severity: | normal | Keywords: | |
Cc: | Trac Release: |
Attachments (2)
Change History (8)
comment:1 Changed 11 years ago by
Reporter: | changed from anonymous to Ryan J Ollos |
---|
Changed 11 years ago by
Attachment: | TracIni.png added |
---|
Changed 11 years ago by
Attachment: | t11036.diff added |
---|
comment:2 Changed 11 years ago by
comment:3 Changed 11 years ago by
Status: | new → assigned |
---|
comment:4 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Thank you for the patch; I appreciate your looking over the code.
Fixed in r13008.
comment:5 Changed 11 years ago by
Sure, no problem. I might have a few other suggestions related to how the markup strings are generated. I'll see if I can find time to take another look.
Have you been working with Trac and Python for long? I'm always curious as to how people are using Trac, particularly plugin authors.
Hint: You can use t:CommitTicketUpdater to have commit messages inserted into ticket. For example, add Refs #11036 or Fixes #11036 in your commit message. You can see some examples in various tickets, such as #10218.
Note: See
TracTickets for help on using
tickets.
Btw, the patch is not quite complete. The placeholders Put some documentation here need to be replaced with real documentation.