Skip to content

New app: Zenhub - #690

Merged
matslina merged 6 commits into
tidbyt:mainfrom
thiagobrez:thiagobrez/zenhub
Oct 6, 2022
Merged

New app: Zenhub#690
matslina merged 6 commits into
tidbyt:mainfrom
thiagobrez:thiagobrez/zenhub

Conversation

@thiagobrez

Copy link
Copy Markdown
Contributor

Integrate your Zenhub workspace to the Tidbyt display.

Zenhub is a productivity management tool to keep track of issues and features like Jira, Asana, Azure DevOps. Using your Zenhub API Keys, we can pull information from your workspace and show in your Tidbyt display.

  • Choose a pipeline to display issues (tickets)
  • Filter by labels
  • Filter by assignees

zenhub

  • In the top row
    • Show the chosen pipeline. Can be any pipeline in your Zenhub workspace. In this example: "Current Sprint"
  • In center
    • Show the first 3 issues (tickets) in the pipeline. Issue number and title

@tidbyt-bot

tidbyt-bot commented Sep 26, 2022

Copy link
Copy Markdown

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@thiagobrez

Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@thiagobrez

Copy link
Copy Markdown
Contributor Author

Included MD5 hash of user's api key in the cache key, per comments in other PRs

@matslina matslina left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey, thanks for the PR!

I've added a few comments in the code, and of course the build is failing but that should be a simple fix I think. When we get it merged, feel free to ping me on discord and I'll get you set up with beta access so you can test it out in production before we publish it. I miiiight set up my own Zenhub account for testing, but maybe not. =)

Comment thread apps/zenhub/zenhub.star
selected_labels = config.str("selected_labels", "")
selected_assignees = config.str("selected_assignees", "")

if not zenhub_rest_api_key:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be great if the app could display an illustrative default graphic when no config is provided. That lets users get a sense of what the app is all about, before they decide to figure out how to grab API keys and such.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey, Mats! Thanks for the review. Nice suggestion, I added a welcome page with fake tasks guiding the user to set-up the configuration info, like below:

welcome_screen.mov

Comment thread apps/zenhub/zenhub.go Outdated
Name: "Zenhub",
Author: "thiagobrez",
Summary: "Show Zenhub issues",
Desc: "Show Zenhub issues from a Pipeline of your choice. Filter by labels and assignees.",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The description fields in get_schema adds a ton of useful context. Unfortunately, these aren't visible to the user in the current Tidbyt iOS/Android app. It would be great if the app description itself could be expanded so users get a good idea of how to set the app up.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I imagined this could be a thing. I moved the guides to the description, see if that works!

@thiagobrez

Copy link
Copy Markdown
Contributor Author

Fixed the build and the other requested changes. Let me know what you think!

@matslina

matslina commented Oct 6, 2022

Copy link
Copy Markdown
Contributor

Looks great, thank you!

@matslina
matslina merged commit 6668ab3 into tidbyt:main Oct 6, 2022
@tidbyt tidbyt locked as resolved and limited conversation to collaborators Oct 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants