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

Add author, image, and read time estimate to blog #481

Merged
merged 20 commits into from
Jun 4, 2022
Merged

Conversation

jaasonw
Copy link
Contributor

@jaasonw jaasonw commented May 24, 2022

Submitting a pull request for once because I'm a blogger now and I now have a personal stake in this part of the website : )

image

image

Also incorporates the changes from #468 but updates the title and test cases

@vercel
Copy link

vercel bot commented May 24, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
acm-csuf-site ✅ Ready (Inspect) Visit Preview Jun 4, 2022 at 8:21AM (UTC)

Cmon guys, it can NOT seriously be this hard to fix this one
Resolves: #430
@vercel vercel bot temporarily deployed to Preview May 24, 2022 06:46 Inactive
oops, I forget karni fixed it, let's use her version instead
@vercel vercel bot temporarily deployed to Preview May 24, 2022 06:52 Inactive
@vercel vercel bot temporarily deployed to Preview May 24, 2022 06:55 Inactive
@jaasonw jaasonw linked an issue May 24, 2022 that may be closed by this pull request
src/routes/blog/index.svelte Outdated Show resolved Hide resolved
Copy link
Owner

@EthanThatOneKid EthanThatOneKid left a comment

Choose a reason for hiding this comment

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

Thanks for getting started on this cool feature! I left some comments for you to consider.

src/lib/blog/util.ts Outdated Show resolved Hide resolved
src/lib/blog/util.ts Outdated Show resolved Hide resolved
src/lib/blog/util.ts Outdated Show resolved Hide resolved
static/global.css Outdated Show resolved Hide resolved
…t in your scss can u space out the nested scss definitions too
@vercel vercel bot temporarily deployed to Preview May 26, 2022 01:13 Inactive
@vercel vercel bot temporarily deployed to Preview May 26, 2022 01:18 Inactive
@vercel vercel bot temporarily deployed to Preview May 26, 2022 01:20 Inactive
@EthanThatOneKid
Copy link
Owner

Could you please rename your utility file before merging this PR to match the rest of the utility files?
/src/lib/blog/util.ts/src/lib/blog/utils.ts
image

Doing this because it's a detail about the implementation that the function caller doesn't actually need to know about but is useful to someone actually reading the implementation
@vercel vercel bot temporarily deployed to Preview May 26, 2022 07:55 Inactive
@vercel vercel bot temporarily deployed to Preview May 26, 2022 09:00 Inactive
@vercel vercel bot temporarily deployed to Preview May 26, 2022 09:01 Inactive
@vercel vercel bot temporarily deployed to Preview May 26, 2022 09:03 Inactive
@vercel vercel bot temporarily deployed to Preview May 27, 2022 00:41 Inactive
@vercel vercel bot temporarily deployed to Preview May 27, 2022 00:53 Inactive
@jaasonw jaasonw enabled auto-merge (squash) May 27, 2022 08:57
@vercel vercel bot temporarily deployed to Preview June 2, 2022 23:22 Inactive
@charliettaylor
Copy link
Contributor

The tags look kinda weird now with the author stuff, maybe add a like Tags: prefix and line it up with the text above it

image

@vercel vercel bot temporarily deployed to Preview June 4, 2022 08:06 Inactive
@jaasonw
Copy link
Contributor Author

jaasonw commented Jun 4, 2022

The tags look kinda weird now with the author stuff, maybe add a like Tags: prefix and line it up with the text above it

image

Resolved by #492

@vercel vercel bot temporarily deployed to Preview June 4, 2022 08:21 Inactive
@jaasonw jaasonw merged commit 450f255 into main Jun 4, 2022
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.

Blog logo unreadable when in dark mode
4 participants