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.

Sign up for GitHub

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

Jump to bottom

Blog Post: Fourth Wave of Nostr Grants #202

Merged
merged 10 commits into from
Dec 21, 2023
Merged

Blog Post: Fourth Wave of Nostr Grants #202

merged 10 commits into from
Dec 21, 2023

Conversation

dergigi
Copy link
Member

@dergigi dergigi commented Dec 21, 2023 •
edited
Loading

To be reviewed by @nvk and sanity-checked.

Build preview:

Social preview:

Screenshot 2023-12-21 at 00 43 41

Live:

@dergigi dergigi self-assigned this Dec 21, 2023
@vercel Vercel
Copy link

vercel bot commented Dec 21, 2023 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
os-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 21, 2023 9:00am

@dergigi dergigi requested a review from nvk December 21, 2023 00:20
@dergigi dergigi marked this pull request as ready for review December 21, 2023 00:44
@dergigi dergigi requested a review from a team December 21, 2023 00:44
Arvin21M
Arvin21M approved these changes Dec 21, 2023
View reviewed changes
Copy link
Contributor

@Arvin21M Arvin21M left a comment

Choose a reason for hiding this comment

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

All the links point to the correct URL. All the content reads fine to me. Noticed an upper case 'N" for Nostr under the Flockstr description, whereas everywhere else its lower case. Did not earmark for edit. Otherwise super excited watching Nostr projects flourish. 🚀

@bitcoinprecept
Copy link

Added minor comment

"A year of experimentation, a year of relaying, a year of building other stuff." is technically not a sentence. Would suggest:

It's been a year of experimentation, a year of relaying, and a year of building other stuff.

Also concur on normalizing whether Nostr is a capitalized proper noun or not.

@Bayernatoor
Copy link
Contributor

Read and Reviewed. Looks good to me.

@dergigi
Copy link
Member Author

dergigi commented Dec 21, 2023

Thanks for the reviews guys. Fixed!

Let's get this baby shipped :shipit:

@dergigi dergigi merged commit 59734a7 into master Dec 21, 2023
2 checks passed
@dergigi dergigi deleted the 4th-wave-nostr branch December 21, 2023 09:06
@dergigi
Copy link
Member Author

dergigi commented May 30, 2024

Live here: https://opensats.org/blog/nostr-grants-december-2023

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Arvin21M Arvin21M Arvin21M approved these changes

@nvk nvk Awaiting requested review from nvk

Assignees

@dergigi dergigi

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@dergigi @bitcoinprecept @Bayernatoor @Arvin21M