FriendsOfFlarum / webhooks

Flarum with outgoing webhooks
MIT License
21 stars 12 forks source link

Fix error when migrating with existing `webhooks.tag_id` foreign #65

Closed dsevillamartin closed 1 year ago

dsevillamartin commented 1 year ago

Fixes #64

Changes proposed in this pull request:

Reviewers should focus on: I think this should work with prefixes... but not entirely sure. Haven't tested that aspect.

Confirmed

jtojnar commented 1 year ago

Can confirm the migration succeeded with 166861ed0d659eed5c4e611347d0233b7f55894a:

Migrating extension: fof-webhooks
Migrated: 2023_06_07_000000_remove_tag_id_constraint
Migrated: 2023_06_07_change_tag_id_to_json
dsevillamartin commented 1 year ago

Thanks for verifying @jtojnar ! I've released v1.2.2 with this fix.