Update Trac instructions. Fix #794. #795
Conversation
This commit doesn't touch trac.rb to avoid breaking existing installations of http://github.com/davglass/github-trac. trac-github is recommended for new installations.
|
Thanks for adding this PR @aaugustin. What do you think about adding a small explanation that like "you shouldn't use this service at all unless you're already using it". Not exactly sure about the copy but I'm trying to figure out how to combat "people don't read" and having a service that you technically shouldn't use. We might also have to introduce the idea of deprecated services or something. Hmm. |
|
That's a good point. I've added a commit to clarify that. |
| 1. **Url** is your Trac install's url | ||
| 2. **Token** (this needs to be the same token you put in your trac.ini installed via the plugin) | ||
| If you're currently relying on this service, please consider upgrading to | ||
| trac-github. It uses a standard Webhook to provide the same features and it's |
There was a problem hiding this comment.
Very, very tiny nit: can you change Webhook to webhook? Trying to keep it a non-proper noun. Then this is good to 🚢!
|
Fixed. I'll also update the spelling in trac-github's docs. |
|
Thanks for all the help |
|
Awesome im learn all this as fast as I can ,u been a big help,really appreciate everything |
Update Trac instructions. Fix #794.
|
Awesome! Thanks @aaugustin for this and letting us know about the old Trac integration. 👍 This will go out later today. |
This commit doesn't touch trac.rb to avoid breaking existing installations of http://github.com/davglass/github-trac. trac-github is recommended for new installations.