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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delete unused post attachments #206

Open
wants to merge 2 commits into
base: staging
Choose a base branch
from

Conversation

jamesli12
Copy link
Contributor

PR Title

Issue Number(s): #181

What does this PR change and why?

Checklist

  • Requirements have been implemented
  • Acceptance criteria are met
  • Database schema docs have been updated or are not necessary
  • Code follows design and style guidelines
  • Commits follow guidelines (concise, squashed, etc)
  • Relevant reviewers (Senior Dev/EM/Designers) have been assigned to this PR

Critical Changes

  • Database change/migration to run
  • Environment config change
  • Breaking API change

Related PRs

Testing

Enumerate steps to test the functionality of your ticket. This should include edge cases and testing of error handling, if applicable.

@jamesli12 jamesli12 requested a review from katsukixyz October 22, 2023 17:26
@netlify
Copy link

netlify bot commented Oct 22, 2023

Deploy Preview for angels-among-us ready!

Name Link
🔨 Latest commit a040b88
🔍 Latest deploy log https://app.netlify.com/sites/angels-among-us/deploys/65355ca15ef7700008f366d6
😎 Deploy Preview https://deploy-preview-206--angels-among-us.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@jamesli12 jamesli12 force-pushed the delete-unused-post-attachments branch from 81168a5 to d31a5a0 Compare October 22, 2023 17:31
@jamesli12 jamesli12 temporarily deployed to development October 22, 2023 17:31 — with GitHub Actions Inactive
@jamesli12 jamesli12 temporarily deployed to development October 22, 2023 17:32 — with GitHub Actions Inactive
@github-actions
Copy link

github-actions bot commented Oct 22, 2023

Coverage report for web

St.
Category Percentage Covered / Total
🔴 Statements 27.98% 465/1662
🔴 Branches 13.28% 64/482
🔴 Functions 15.24% 75/492
🔴 Lines 23.67% 343/1449

Test suite run success

40 tests passing in 5 suites.

Report generated by 🧪jest coverage report action from a040b88

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.

1 participant