thisdot / starter.dev

A collection of kits to help bootstrap your next project.
https://starter.dev
MIT License
127 stars 16 forks source link

Delay notification and account for users with special character in thank you GH action #1308

Closed jdwilkin4 closed 1 year ago

jdwilkin4 commented 1 year ago

There are two issues that this PR should solve

Issue No.1:

The notification when a PR is merged shows up pretty quickly. We want to delay the thank you notification by a couple of minutes.

Issue No.2:

While testing the GH action earlier, we noticed that when the curl command was run to check if a user had previously contributed, it would throw an error if the user had special characters in their username. We need to add a condition to account for that so the action doesn't fail.