klothoplatform / slack-notifier

Slack notification bot
MIT License
3 stars 0 forks source link

bot doesn't correctly track PRs across org name changes #52

Open ghost opened 1 year ago

ghost commented 1 year ago

Let's say you had a repo at Acme/foo-repo, with a PR 123. If you rename Acme to WidgetMakers, then the bot loses track of the PR, and won't update its thread. It will just ignore all updates to that PR.

This is because we identify each PR by its URL, which includes the org name (afaik, there isn't any global PR identifier). When the org name changes, we can't find the original thread that the bot made, and without that it can't update or reply to it.

I believe this would apply to repo name changes too, though I haven't tested that.