-
-
Notifications
You must be signed in to change notification settings - Fork 39
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
New Formatting Options #95
Comments
I talked to @TkTech a while ago about implementing a custom template system for messages, I think this would satisfy all needs for custom formatting. But I am not sure if the idea is still relevant or is already thrown away. |
It is still on the list, but a templating system is hard to sandbox without neutering it. Jinja2 provides a sandbox, but is too easy to lock up (infinite for loop, infinitely growing templates, etc...) |
Yay Sauerbraten :) Anyway, I came here looking for almost this exact thing Gemrb are using an irc to discord bridge to keep messages flowing between them and currently the github notifications are drowning out the chat because discord makes a preview link for every url that's posted. In discord it can be prevented by adding <> brackets around the url, but it would take having this sort of optional template formatting to make it happen. .... so.... uhh.... Is this still on the list? :) |
Additionally to #94 i would love to see some more formatting options for the irc-bot in general:
3 lines for the simple message "Success" can be optimized i think.
In my opinion one line like:
is totally enough.
Maybe even without the details-link (because you can find it quite easily anyways) or with short-urls
so
"https://travis-ci.org/sauerbraten-fork/sauerbraten-fork/builds/35527292" becomes something like "https://travis-ci.org/sMlthrG/"
The text was updated successfully, but these errors were encountered: