Skip to content
This repository has been archived by the owner on Oct 15, 2022. It is now read-only.

Convert Crontab cheatsheet to JSON #1186

Closed
wants to merge 2 commits into from

Conversation

alfonzm
Copy link
Contributor

@alfonzm alfonzm commented Jun 14, 2015

This is my attempt at converting the Crontab cheat sheet into the new JSON format (in reference to #1155).

Screenshots:
screen shot 2015-06-14 at 11 55 02 pm
screen shot 2015-06-14 at 11 54 49 pm

@javathunderman
Copy link
Collaborator

cc @jagtalon

@moollaza moollaza self-assigned this Jun 15, 2015
@moollaza
Copy link
Member

/cc @abeyang

This one looks OK, but the content is a bit of an edge-case

Also, I just noticed that I need to add some CSS to maintain the width of the description in the expanded state :)

@moollaza
Copy link
Member

Also, @alfonzm thanks a lot for contributing, we really appreciate it! We'll be sure to review this shortly and give you some feedback!

@zachthompson
Copy link
Contributor

When I looked at the old one initially it looked challenging to convert. We might consider remove all /path/to/command entries to slim it down since we're really discussing date/time specifications. Maybe a single entry under fields explaining that they all take a command to execute.

@moollaza
Copy link
Member

@zachthompson that's a great suggestion.

@alfonzm let's remove the path/to/command from the examples.

"key" : "[1-12] or [first 3 letters]",
"val" : "month"
}, {
"key" : "[0-7] or [first 3 letters (0 or 7 is Sun)]",
Copy link
Member

Choose a reason for hiding this comment

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

Let's move the "(0 or 7 is Sun)" to the value

@mintsoft
Copy link
Collaborator

mintsoft commented Aug 7, 2015

@alfonzm Just checking in, are you still interested in finishing this off?

"metadata": {
"sourceName": "Crontab Cheat Sheet",
"sourceUrl": ""
},
Copy link
Member

Choose a reason for hiding this comment

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

We've just added support for CheatSheet Templates that change the layout to favour certain kinds of data. Let's add "template_type": "terminal" to this IA. It should make the keys much easier to read!

@moollaza
Copy link
Member

@alfonzm Ping! I'm just checking in to see if you're still interested in finishing this one up? We'd love to see this go live!

There's absolutely no rush, just wanted to make sure you're not stuck on anything.

@zekiel
Copy link
Member

zekiel commented Sep 30, 2015

this will use the current IA page here: https://duck.co/ia/view/crontab_cheat_sheet

@GuiltyDolphin
Copy link
Member

@alfonzm @moollaza @zachthompson Situation with this one? I'd like to see this (and any other non-JSON cheat sheets) converted so they stay up-to-date with any changes to the controller.

@mintsoft
Copy link
Collaborator

mintsoft commented Feb 6, 2016

@GuiltyDolphin I'm marking this as abandoned and closing it; if you want to pick it up feel free

@mintsoft
Copy link
Collaborator

mintsoft commented Feb 6, 2016

@alfonzm I've marked this as abandoned, if you'd like to finish it up etc feel free to ping us and we can reopen it for you :)

@javathunderman
Copy link
Collaborator

@moollaza seems like #1155 has been closed - the Abandoned label can be removed.

@GuiltyDolphin
Copy link
Member

@javathunderman Looks like it hasn't been moved over yet! I'll leave it on until the Goodie has been ported. (I think I had a branch somewhere...)

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

Successfully merging this pull request may close these issues.

None yet

7 participants