Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃彈 Dropped webhooks.status column #14713

Merged
merged 2 commits into from
May 11, 2022
Merged

馃彈 Dropped webhooks.status column #14713

merged 2 commits into from
May 11, 2022

Conversation

daniellockyer
Copy link
Member

@daniellockyer daniellockyer commented May 5, 2022

refs https://github.com/TryGhost/Toolbox/issues/309

  • this column is not used and I was going to add validation to it but
    it's better to clean it up and re-add the column if we need it again

@daniellockyer daniellockyer added the running-ci-checks PR's which do not need/want review label May 5, 2022
@daniellockyer daniellockyer self-assigned this May 5, 2022
@codecov
Copy link

codecov bot commented May 5, 2022

Codecov Report

Merging #14713 (930a7f3) into main (412d290) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main   #14713      +/-   ##
==========================================
- Coverage   61.15%   61.14%   -0.02%     
==========================================
  Files         572      572              
  Lines       46841    46839       -2     
  Branches     4207     4208       +1     
==========================================
- Hits        28646    28640       -6     
- Misses      18151    18155       +4     
  Partials       44       44              
Impacted Files Coverage 螖
core/server/data/schema/schema.js 100.00% <酶> (酶)
core/server/models/base/plugins/events.js 69.48% <0.00%> (-1.48%) 猬囷笍

Continue to review full report at Codecov.

Legend - Click here to learn more
螖 = absolute <relative> (impact), 酶 = not affected, ? = missing data
Powered by Codecov. Last update 412d290...930a7f3. Read the comment docs.

@TryGhost TryGhost deleted a comment from github-actions bot May 9, 2022
@daniellockyer daniellockyer marked this pull request as ready for review May 9, 2022 12:33
@daniellockyer daniellockyer removed the running-ci-checks PR's which do not need/want review label May 9, 2022
@daniellockyer daniellockyer changed the title Added validation to webhooks.status column 馃彈 Dropped webhooks.status column May 11, 2022
@daniellockyer
Copy link
Member Author

@matthanley + @SimonBackx: note that I've changed the migration to dropping the webhooks.status column

refs TryGhost/Toolbox#309

- this column is not used and I was going to add `validation` to it but
  it's better to clean it up and re-add the column if we need it again
@daniellockyer daniellockyer merged commit 84a10da into main May 11, 2022
@daniellockyer daniellockyer deleted the webhooks-status branch May 11, 2022 13:11
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.

None yet

3 participants