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

correcting typo, compare to ch2 from the book #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bahunov
Copy link

@bahunov bahunov commented Feb 2, 2022

Adding picture to the matrix,
Correcting typo directly from the book:
https://learning.oreilly.com/library/view/clean-architecture-a/9780134494272/ch2.xhtml#toclev_8

@bahunov
Copy link
Author

bahunov commented Feb 2, 2022

@preslavmihaylov please review

@preslavmihaylov
Copy link
Owner

preslavmihaylov commented Feb 7, 2022

Actually, the quadrants are correct based on their indices in plane geometry.

But also, the redacted part doesn't preserve the original meaning:

  • Architecture is actually only in quadrant Important, Not Urgent, rather than Important, Urgent

  • also:

This way, the important bits of quadrant 1 (Important, Urgent) never get the proper attention.

Actually, the I, NU bits are the ones which get neglected. I, U usually get addressed.

@bahunov
Copy link
Author

bahunov commented Feb 8, 2022

@preslavmihaylov I think you're mistaken, read the chapter in the book again... it doesn't seem to be based on plane geometry:
"Of course, some things are both urgent and important. Other things are not urgent and not important. Ultimately, we can arrange these four couplets into priorities:

  1. Urgent and important

  2. Not urgent and important

  3. Urgent and not important

  4. Not urgent and not important"

Compare with my picture

@preslavmihaylov
Copy link
Owner

alright, regardless of the quadrants format, I think your picture makes it easier to understand based on the indices you've added so I'll accept that.

How about the other concerns though?

But also, the redacted part doesn't preserve the original meaning:

  • Architecture is actually only in quadrant Important, Not Urgent, rather than Important, Urgent
  • also:

This way, the important bits of quadrant 1 (Important, Urgent) never get the proper attention.

Actually, the I, NU bits are the ones which get neglected. I, U usually get addressed.

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

Successfully merging this pull request may close these issues.

2 participants