-
Notifications
You must be signed in to change notification settings - Fork 35
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
Create blog.md #228
base: master
Are you sure you want to change the base?
Create blog.md #228
Conversation
data visualization blog commit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good! left a number of inline comments. highlighting one in the middle with formatting since it applies to a number of places i didn't highlight:
just started to notice a handful of [screenshots on the same line as text] throughout the file, do a run through end to end to make sure there's a blank line before every screenshot so the formatting stays consistent
Added Tableau download link, public Tableau dashboard link, and a few more screenshots.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks great! The writing team will take a final pass in a few weeks and we should be good to publish your blog!
data visualization blog commit
Reference to issue
Description of the changes proposed in the pull request
Reviewers requested: