-
Notifications
You must be signed in to change notification settings - Fork 6
NgRx Data Views #4
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
Conversation
Didn't take a closer look yet, but: Wow! Thank you already, I will provide feedback soon. |
No need to change this now, but for the next post: When you put each sentence in a new line, it's easier to review, comment and merge with Git 🙂 |
Wow, I'm impressed! This is exactly as I imagined. For my taste there could be some more elaborate explanations but this is all about who the target audience is. Therefore, it's important to provide guidance at the very beginning and tell the reader what we will talk about and what they should already know. What you should add is the classification of the idea: Is it just a laboratory idea or is it something you use in a big enterprise app? Something like:
This makes a lot more appreciation towards the idea, because people see: "Wow, this is not just some nonsense but those folks tested this in production." When it comes to me the post should have a few more pictures, however I don't know where yet. 😅 Thanks so far! This is going to be a very good article! // cc @JohannesHoppe |
33d4b96
to
15040d0
Compare
@dhhyi I rebased, squashed and merged into |
@dhhyi |
Did some minor proof reading (see 0d53186), squashed your commits and merged them into @JohannesHoppe and I discussed the release date and we agreed that we should release this in the first week of January to avoid the xmas fatigue. I will create the header picture before this. |
Preview here: https://angular.schule/blog/2020-01-ngrx-data-views |
About the cover picture. I would highlight the aspect "other view on the data" or "layers of data". |
Now imagine the second image with some cool bold white text on the grey ground. 😍 |
@JohannesHoppe I would REALLY like to see this background: https://www.dropbox.com/s/6vnddw97x2h8h1e/IMG_20180910_081910.jpg?dl=0 Because it is subtle and inspirational. Dare to overrule me ;P |
Some leaves. I do not see any relation to the topic?? 🤨 |
@JohannesHoppe Okay, you're right, there's no direct relation to the topic, other than the leaves giving a blurry glimpse (read "view") of my working place. I thought it would be a nice greeting to my fellow workers, if they recognize it while stumbling upon this article. Also I took that photo and thereby can relate to it. 😉 |
Not my personal favourite (I'd prefer a more technical picture as well) but I could live with it ;-) |
😕 |
Awesome. Looks great. Thanks 🙂 👍 |
One thing I still forgot: Fix the "NEW" label in the demo application so it marks the latest book as new in 2020. 😅 I'll be right on it. |
Done 👍 |
You updated the Stackblitz example only and not the text, right? So nothing to do for me here? |
Right. Now everything is working again and if you want you can fork the examples on StackBlitz. You can also leave it like it is now, I won't delete the demo repo 😉 |
Hi folks,
@fmalcher said I could write if I felt like it - turns out I did 😁
Please provide feedback, thx,
Danilo