community-platform
community-platform copied to clipboard
[DRAFT] fix: update user delete procedure
trying a function that triggers when the auth record is deleted to get round permission denied error
#1412
PR Checklist
- [ ] - Commit messages are descriptive, it will be used in our Release Notes
PR Type
- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)
- [ ] Developer experience (improves developer workflows for contributing to the project)
Description
What this PR does
Git Issues
Closes #
Screenshots/Videos
If useful, provide screenshot or capture to highlight main changes
What happens next?
Thanks for the contribution! We try to make sure all PRs are reviewed ahead of a monthly dev call (first Monday of the month, open to all!).
If the PR is working as intended it'll be merged and included in the next platform release, if not changes will be requested and re-reviewed once updated.
If you need more immediate feedback you can try reaching out on Discord in the Community Platform development
channel.
Visit the preview URL for this PR (updated for commit 01340f0):
https://onearmy-next--pr2419-1412-bug-deleting-us-b6ntn8xh.web.app
(expires Thu, 13 Jul 2023 15:47:10 GMT)
🔥 via Firebase Hosting GitHub Action 🌎
Sign: 6d65e4f8fee2f6ab2da0c1c3b85b8797d66afa59
Codecov Report
Patch coverage has no change and project coverage change: -0.24
:warning:
Comparison is base (
fbe1b3d
) 44.65% compared to head (01340f0
) 44.41%.
Additional details and impacted files
@@ Coverage Diff @@
## master #2419 +/- ##
==========================================
- Coverage 44.65% 44.41% -0.24%
==========================================
Files 253 253
Lines 6873 6871 -2
Branches 1525 1525
==========================================
- Hits 3069 3052 -17
- Misses 3775 3787 +12
- Partials 29 32 +3
Impacted Files | Coverage Δ | |
---|---|---|
src/stores/User/user.store.ts | 1.89% <ø> (+0.02%) |
:arrow_up: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
2 flaky tests on run #3605 ↗︎
![]() |
![]() |
![]() |
![]() |
![]() |
Details:
fix: update user delete procedure | |||
Project: onearmy-community-platform | Commit: 01340f04c5 |
||
Status: Passed | Duration: 04:32 💡 | ||
Started: Jun 13, 2023 3:49 PM | Ended: Jun 13, 2023 3:54 PM |
research/write.spec.ts • 1 flaky test • ci-chrome
Test | Artifacts | |
---|---|---|
[Research] > [Edit a research article] > [By Owner] |
Output
Screenshots
|
map.spec.ts • 1 flaky test • ci-chrome
Test | Artifacts | |
---|---|---|
map > [By User] > should show the user a message stating their pin is rejected |
Output
Screenshots
|
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.
Closing as approach didn't work