-
Notifications
You must be signed in to change notification settings - Fork 25
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
Add a new card from the widget #37
Comments
Here's my proof-of-concept for the add button. I actually have it fully coded and working: Features:
|
An interesting side-effect of this - I had to change the authentication a bit, as to do this kind of thing we need to start requesting write privileges. I have this implemented and working, too, to a point. If you upgrade from a version that only has read privs, you need to logout and re-login to enable the new privs. If you click "add" and try to add a card before you've done this login/logout, the only error you get right now is a toast that tells you to login and logout again... Maybe this can be made better. |
I just reworked my change some, so that when you click the "add" button but we don't have write privs yet, it pops up a dialog: Clicking "Cancel" just closes the dialog, and the widget functions as before (the add button can be hidden via on option screen toggle) Clicking "OK" invalidates our cached login token and opens up the main activity where clicking the "Login" button will now request write access. |
FYI: Work-in-progress branch is here: https://github.com/lack/TrelloWidget/tree/add_button |
This is pretty cool.
I'll have to say no to this, even though it looks like you've put a decent amount of work into it. |
I'd love a "quick add" button in the title-bar of the widget. It could open a little dialog that lets me add a card (by title only). The goal would not be to re-create trello's whole "add card" with all the features and additions and things, but just a quick thing for one-off ideas that I would then edit in trello later as I refined them.
The text was updated successfully, but these errors were encountered: