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

Checklists #11

Merged
merged 8 commits into from
Jul 2, 2019
Merged

Checklists #11

merged 8 commits into from
Jul 2, 2019

Conversation

nelsonic
Copy link
Member

@nelsonic nelsonic commented Apr 30, 2019

My goal with this pull request is to:

@nelsonic nelsonic added the in-progress An issue or pull request that is being worked on by the assigned person label Apr 30, 2019
@nelsonic nelsonic self-assigned this Apr 30, 2019
@nelsonic
Copy link
Member Author

nelsonic commented May 1, 2019

This is the Post-it Prototype of my "Morning Routine" checklist that I follow each morning:
nelson-mourning-routine-checklist-postit-prototype

Reduced image dimensions and Optimised:
nelson-mourning-routine-checklist-postit-prototype-min
Slight loss of quality on text but 95% reduction in file size!

@nelsonic nelsonic marked this pull request as ready for review June 9, 2019 21:48
@nelsonic nelsonic added awaiting-review An issue or pull request that needs to be reviewed and removed in-progress An issue or pull request that is being worked on by the assigned person labels Jun 9, 2019
@nelsonic
Copy link
Member Author

nelsonic commented Jun 9, 2019

@iteles please review & merge when you can.
Thanks. 🌻

checklists.md Outdated
And in doing this we will _systematically_:
1. Build a tool that we will use ourselves to:
1. maximise our _personal_ effectiveness
2.
Copy link
Member

@SimonLab SimonLab Jun 10, 2019

Choose a reason for hiding this comment

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

I don't think the indentation is correct on this list:
image

Copy link
Member Author

Choose a reason for hiding this comment

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

@SimonLab good spot. I've added the missing bullet point.

checklists.md Outdated
> "_a list of items required, things to be done,
or points to be considered, used as a reminder._"

## The Checklist Manifesto
Copy link
Member

Choose a reason for hiding this comment

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

Looks like this subtitle is not rendered properly:
image

checklists.md Outdated

# _Why?_

## Why does this _matter_?
Copy link
Member

Choose a reason for hiding this comment

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

Subtitle rendering issue:
image

Copy link
Member Author

Choose a reason for hiding this comment

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

@SimonLab thanks for taking the time to read the PR and make these comments.
I think I've resolved the Markdown issues.

@nelsonic nelsonic assigned iteles and unassigned nelsonic Jun 22, 2019
@nelsonic
Copy link
Member Author

@iteles I've made the amendments suggested by @SimonLab (12 days ago). 📝
Please review and merge when you can. 👀
Thanks. ☀️

Copy link
Member

@iteles iteles left a comment

Choose a reason for hiding this comment

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

Oops. My apologies, I don't know how I managed to miss this 😞

@iteles iteles merged commit 0a74a2b into master Jul 2, 2019
@iteles iteles deleted the checklists-issue#9 branch July 2, 2019 12:26
@nelsonic
Copy link
Member Author

nelsonic commented Jul 3, 2019

@iteles thanks for merging. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-review An issue or pull request that needs to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants