-
Notifications
You must be signed in to change notification settings - Fork 4
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
Implement /newtask
#32
Comments
Should be inlined "inline function" as we discussed before. |
I spent an hour reading through inline docs etc last night to try implement repo name autocomplete. https://core.telegram.org/bots/api#inline-mode I think you want to type
I'll attempt to implement it, showing why it cannot be done if that's the case and we'll take it from there. |
This is the trouble with the inline query; it's that we don't know which comment you are referring to even when direct replying and then mentioning the bot as the update doesn't reference it.
The above means that somehow we'd need to capture the original message and then hoist that into the inline_query update and then persist that through the chain of inline_query updates as we need to select which org then which repo. So at least 3 inline_queries to chain after capturing the task inspo comment. There are advanced parts of Grammy that allow branching and forking of different ctx and updates and splicing them but that's beyond reach atm, still a lot of learning involved with the APIs and the libs we are using. Two brand new APIs and two libs; one for MTProto, one for BotAPI. Let's ship the feature as a command for V1 and a task can be dedicated to upgrading it to an 'inline function', as it'll require more research and quite a bit of work as mentioned above. Do you agree? |
A new admin controlled command that allows the creation of GitHub tasks direct from telegram.
We lack the same context that we do on GitHub so it's required that the invoker pass in the
repository.full_name
(ubiquity-os-marketplace/ubiquity-os-kernel-telegram) so that we deliver the issue to the correct repo. This makes sense as admins will know immediately where they'd want it opened.The text was updated successfully, but these errors were encountered: