Skip to content

Improve grantee report submission and bot integration (take 2)#433

Closed
BoltTouring wants to merge 553 commits into
masterfrom
pr-430
Closed

Improve grantee report submission and bot integration (take 2)#433
BoltTouring wants to merge 553 commits into
masterfrom
pr-430

Conversation

@BoltTouring
Copy link
Copy Markdown
Contributor

@BoltTouring BoltTouring commented Apr 1, 2025

This PR creates the grantee report submission feature and fixes several issues with the bot integration and error handling.

Changes

  • Added better logging throughout the report submission process for easier debugging
  • Improved error handling with a new ErrorBoundary component
  • Fixed TypeScript and ESLint issues across multiple components
  • Added proper validation for report submissions
  • Improved email template and confirmation handling

Testing

The changes have been tested locally with:

  • Multiple successful report submissions to test issue
  • Proper bot token usage confirmed (posting as OpenSatsBot)
  • Email confirmations working correctly
  • Error boundary catching and displaying errors appropriately

Logs from testing

Successfully posted comment to GitHub: https://github.com/OpenSats/reports/issues/231#issuecomment-2770694240
Email sent successfully to rob@opensats.org in 1135ms

lucasmogu and others added 30 commits January 17, 2024 13:28
Remove "Project Listing" Wording
* Add 1st draft

* Add link to https://ln.rene-pickhardt.de/

* Reword & add links to BitMEX

* Remove renepay

* Link to StackOverflow question

* Fix publication date

* Add cover image
Very minor changes, feel free to merge after review @lucasmogu
* 1st draft of 0xB10C blog post

* fix: grantee feedback

* fix: feedback from grantee

* fix: authors

* feat: add cover image

* feat(post): use nym instead of twitter handle

* fix: date

* fix: authors
Co-authored-by: Lucas <81401034+lucasmogu@users.noreply.github.com>
Also made a small change to the volunteer bios. Clicking on "Volunteer"
now brings you to
[/about#volunteers](https://opensats.org/about#volunteers).
Change bio from Chief Fiat Officer to Volunteer
* Addison -> Ace

* Rename files
* Update j9.mdx to replace broken URL for CBDC Tracker

J9's bio on the opensats.org site has a hyperlink associated with the "CBDC Tracker Fellowship" words. This hyperlink is broken (https://cbdchumanrights.org/). It needs to be replaced with the following (https://cbdctracker.hrf.org/home) in order to direct viewers to the CBDC Tracker page hosted by HRF.

* Update j9.mdx

Original broken link was:
https://cbdchumanrights.org/

Initially proposed changing to:
https://cbdctracker.hrf.org/home

But better to point to the fellowship link here:
https://cbdctracker.hrf.org/fellows
* Initial import

* Fix markdown violations

* Insert ToC and donate button

* Insert "Donate to Ops" button

* Formatting and line-breaks

* More formatting

* Fix title and date

* Fix author and project order

* Use relative links

* Link to dni profile

* Add cover image

* space -> ecosystem
Merging as requested
dergigi and others added 27 commits March 6, 2025 19:40
* Introduce "Reports & Statistics" section

* Add link to impact reports
* Skeleton

* Add 1st Draft

* Add Header Image

* Update image file name in skeleton

* Add parenthesis

* add mailto per Bayer

Co-authored-by: Bayer <55212954+Bayernatoor@users.noreply.github.com>

* Add missing punctuation

* Update grant project licenses

* Move paragraph up

* Convert to bullet point list

* Minor edits

* Update publication date

* Remove advertisement language

* Uppercase -> lower case

* Edit url destination

Co-authored-by: Amiti Uttarwar <amiti@uttarwar.org>

* Small phasing edit

Co-authored-by: Amiti Uttarwar <amiti@uttarwar.org>

* Update publication date

---------

Co-authored-by: Bayer <55212954+Bayernatoor@users.noreply.github.com>
Co-authored-by: Gigi <dergigi@pm.me>
Co-authored-by: Amiti Uttarwar <amiti@uttarwar.org>
Add PGP Public Key + Fingerprint
* Skeleton

* Add 1st Draft

* Update header image file name in skeleton

* Upload header image file

* Adjust date to not be in the future>>to enable build preview

* Update date again>>2nd try at getting build preview using date

* Fix filename (missing .mdx)

* Edit: was >> has been

Co-authored-by: Jon Atack <jon@atack.com>

* Update date & author field

---------

Co-authored-by: Gigi <dergigi@pm.me>
Co-authored-by: Jon Atack <jon@atack.com>
* Add two new items to website faq/grantee

Q: What should I do if I feel overwhelmed while working on my projects?

If you're feeling overwhelmed, reach out to us at support@opensats.org and let us know. Alternatively, our grantee [Waye](https://www.waye.dev/about) offers structured support to help developers manage stress and avoid burnout through programs with group discussions, coaching, and reflection tools. You can reach them at [waye.dev@gmail.com](mailto:waye.dev@gmail.com)

Q: Can you help set up a dedicated non-profit foundation for my project?

A: OpenSats does not provide this service, but our grantee, the [Foundation Formation Kit](https://docs.foundationformationkit.org/), offers guidance and documentation on setting up a non-profit for open-source projects.

* Copy edits

* Update ToC

---------

Co-authored-by: Gigi <dergigi@pm.me>
- Create complete report submission flow for OpenSats grantees
- Integrate SendGrid for confirmation emails with report copies
- Implement GitHub integration for report submission as issue comments
- Add development mode support with test grant IDs
- Create documentation for setup and testing
…SLint issues, add error handling, improve email template and validation, fix bot token usage
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 1, 2025

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

Name Status Preview Comments Updated (UTC)
os-website ❌ Failed (Inspect) Apr 1, 2025 11:30pm

@BoltTouring
Copy link
Copy Markdown
Contributor Author

BoltTouring commented Apr 2, 2025

Closing in favor of #434

@BoltTouring BoltTouring closed this Apr 2, 2025
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.

10 participants