Skip to content

Tag application coming from Common Grant app #446

Merged
dergigi merged 4 commits into
OpenSats:masterfrom
jrakibi:05-04-common-app-label
Jun 9, 2025
Merged

Tag application coming from Common Grant app #446
dergigi merged 4 commits into
OpenSats:masterfrom
jrakibi:05-04-common-app-label

Conversation

@jrakibi
Copy link
Copy Markdown
Contributor

@jrakibi jrakibi commented May 5, 2025

This PR tags applications coming from the common grant app and adds data about the organizations the grantee applied to

  • Added a new label 'common-grant-app' in github issues to identify applications submitted through the common grant application
  • Added a new section "Organizations Applied To" in the GitHub issue body to display which organizations the grantee has applied to

@vercel
Copy link
Copy Markdown

vercel Bot commented May 5, 2025

@jrakibi is attempting to deploy a commit to the OpenSats Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 5, 2025

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 May 22, 2025 8:56pm

Comment thread pages/api/github.ts Outdated

${req.body.potential_impact}

### Organizations Applied To
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
### Organizations Applied To
### Other Organizations Applied To

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

@dergigi
Copy link
Copy Markdown
Member

dergigi commented May 5, 2025

There seems to be a prettier issue, you should be able to reproduce it locally if you run npm run build yourself @jrakibi - you should also see it in your IDE as a linter warning (and you should be able to auto-fix it easily).

@jrakibi
Copy link
Copy Markdown
Contributor Author

jrakibi commented May 6, 2025

There seems to be a prettier issue, you should be able to reproduce it locally if you run npm run build yourself @jrakibi - you should also see it in your IDE as a linter warning (and you should be able to auto-fix it easily).

Thanks @dergigi, fixed. Should be working fine now.

Copy link
Copy Markdown
Member

@dergigi dergigi left a comment

Choose a reason for hiding this comment

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

Apologies for the delay on this. Merging now!

Looks good to me ✅

@dergigi dergigi merged commit 419f4df into OpenSats:master Jun 9, 2025
3 checks passed
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.

2 participants