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

Added a simpler template just for cheat sheets #1245

Merged
merged 3 commits into from
Jul 8, 2015

Conversation

tagawa
Copy link
Collaborator

@tagawa tagawa commented Jul 6, 2015

If this is approved then I'll add a link to the DDH documentation.

## Checklist
Please place an 'X' where appropriate.

```
Copy link
Member

Choose a reason for hiding this comment

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

I think we can probably remove this section? We're controlling the UI so it will look the same on all devices.

@moollaza
Copy link
Member

moollaza commented Jul 6, 2015

/cc @talsraviv -- Any thoughts on this?

@talsraviv
Copy link
Contributor

Agreed - a lot of things about cheat sheets are not unique so a lot of these can be taken out. Good idea.

@tagawa
Copy link
Collaborator Author

tagawa commented Jul 7, 2015

Cool, that makes it even simpler!

**What does your Instant Answer do?**


**What is the data source for your Instant Answer? (Provide a link if possible)**
Copy link
Contributor

Choose a reason for hiding this comment

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

@moollaza @tagawa I was thinking, are the data source questions relevant to cheat sheets? Maybe another opportunity to simplify.

Copy link
Member

Choose a reason for hiding this comment

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

Source is still important I'd say, we do link to whatever source you provide in the JSON. And there are definitely other sources that may have more info. E.g. Adobe.com for Photoshop vs http:https://digital-photography-school.com/photoshop-shortcut-commands/

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I agree, I think source is still relevant, including for when/if there are complaints about the correctness of facts within a cheat sheet and we need to verify.

@talsraviv
Copy link
Contributor

Cool - makes sense.

@tagawa already noted this above - guessing the last step after merging is adding this to the list at https://duck.co/duckduckhack/submission_and_review

@moollaza this looks good to me... merge?

@tagawa
Copy link
Collaborator Author

tagawa commented Jul 8, 2015

Just sent the PR for the documentation link: duckduckgo/duckduckgo-documentation#295

moollaza added a commit that referenced this pull request Jul 8, 2015
Added a simpler template just for cheat sheets
@moollaza moollaza merged commit a68d503 into duckduckgo:master Jul 8, 2015
@tagawa
Copy link
Collaborator Author

tagawa commented Jul 8, 2015

Great, thank you @moollaza and @talsraviv

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

4 participants