Skip to content
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

Validation #5

Open
scottwarren opened this issue Dec 4, 2017 · 2 comments
Open

Validation #5

scottwarren opened this issue Dec 4, 2017 · 2 comments
Labels

Comments

@scottwarren
Copy link

scottwarren commented Dec 4, 2017

Hey!

I'd love to help out with this project, and I found this comment:

overtime-cli/index.js

Lines 18 to 19 in b56fac7

// TODO: BETTER VALIDATION
assert(args.regions.length > 0)

What are your thoughts/plans/dreams for the validation? I would love to contribute in a meaningful and helpful way 👍

@diit
Copy link
Owner

diit commented Dec 4, 2017

Hey @scottwarren thanks for reaching out!

So for the next version, take a look at the current PR's, rough roadmap is as follows

  1. Remove vorpal, and any unnecessary packages, as long as it doesn't increase the difficulty for new devs to contribute.
  2. Store local aliases and working hours, use
    overtime show America/Toronto@me Europe/Budapest@JohnDoe
    for header aliases, and then be able to use them later
    overtime show @me @JohnDoe
    (This combines two of the outstanding PR's)
  3. Slack integration, I have a version of this (around Jimp) that outputs it as an image, so building it into a slack bot that pulls do not disturb and timezone details from people in the room, or passed to bot would be the ideal use case.

@diit diit added the question label Dec 4, 2017
@scottwarren
Copy link
Author

Thank you! My work might be a little slow, but I'd like to help with one of these tasks for sure. I'll take a look at the open PR's.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants