search

snowplow / snowplowanalytics.com

The Snowplow website
https://snowplowanalytics.com
12 stars 19 forks source link

First TMA-2 blog post draft #1336

Closed chuwy closed 4 years ago

chuwy commented 4 years ago

Hey @benjben, @mikenem! I think I'm done with most technical parts of the blog post and generally feel that structure is right, but wording is horrible.

chuwy commented 4 years ago

Agree with everything @benjben mentioned. @mikenem are you okay to fix that on your pass or you want me to edit it again?

mikenem commented 4 years ago

hey @chuwy @benjben , I've gone in and edited as well as added most of Ben's comments. Please do have a final look through and then we can merge to next and have Lyuba look through it.

benjben commented 4 years ago

Looks great @mikenem !

I wonder if we should just always use failed events every where instead of bad rows. But the library is named snowplow-badrows.

mikenem commented 4 years ago

Yeah it's a tough call. I think eventually we want everything to be failed events. But since so many people still know it as bad rows... I'm trying to slowly merge things over. Looks like when you rename a repo GH automatically redirects requests to the new URL. But maybe we hold that off until the next update to snowplow-badrows

mikenem commented 4 years ago

@yalisassoon @goodits let me know if you have any edits and I'll change before pushing to next