Skip to content

feat!: dismiss from reschedule confirmations #73

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

Merged
merged 43 commits into from
Apr 25, 2025

Conversation

williscool
Copy link
Owner

@williscool williscool commented Apr 10, 2025

TODO

  • I would exclude reccuring events from this its hard to tell if you are dismissing the right one (we look ahead a year and another part of the codebase always picks the latest event so that could mean we dissmiss based on a future version of a reccuring event) will handle in Fast Follows: we need to exclude recurring events from Reschedule Confirmations dismissals #81
  • we should documetn (called it ! lol alter publication powersync add table public.reschedule_confirmations)

  • feat: dismiss from reschedule confirmations!
  • feat: ids array over the bridge!
  • feat: json over the bridge!
  • feat: stub for in the app
  • feat: events all the way to the app side!
  • feat: maybe dismiss from react native!
  • feat: updated dissmiss message
  • feat: safe dismiss in bulk
  • test: tests!
  • chore: interface
  • test: event dissmiss tests run
  • test: ignore orig tests for now

@williscool williscool changed the title feat/dismiss from reschedule confirmations feat!: dismiss from reschedule confirmations Apr 10, 2025
Copy link

Build artifacts for this PR are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

Build artifacts for this PR are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

github-actions bot commented Apr 11, 2025

Code Coverage Report

Overall Project 13.77% -1.19%
Files changed 28.66%

Module Coverage
app 26.25% -0.72%
x8664Debug 1.43% -1.66%
Files
Module File Coverage
app EventDismissResult.kt 90.79% -9.21% 🍏
DismissedEventAlertRecord.kt 57.69% -12.5% 🍏
ApplicationController.kt 37.22% -8.19%
RescheduleConfirmationsBroadcastReceiver.kt 0%
DismissedEventListAdapter.kt 0% -3.04%
x8664Debug DismissedEventListAdapter.kt 0% -3.04%
RescheduleConfirmationsBroadcastReceiver.kt 0%
ApplicationController.kt 0% -19.55%
EventDismissResult.kt 0%
DismissedEventAlertRecord.kt 0% -37.5%

Copy link

📊 Code Coverage Summary

Type Coverage
Line 13.53
Branch

View detailed coverage report

Copy link

Build artifacts for this PR are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

📊 Code Coverage Summary

Type Coverage
Line 13.52
Branch

View detailed coverage report

Copy link

📊 Code Coverage Summary

Type Coverage
Line 13.49
Branch

View detailed coverage report

Copy link

Build artifacts for this PR are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

📊 Code Coverage Summary

Type Coverage
Line 13.98
Branch

View detailed coverage report

Copy link

Build artifacts for PR #73 (commit 932424f) are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

📊 Code Coverage Summary

Type Coverage
Line 13.87
Branch

View detailed coverage report

Copy link

Build artifacts for PR #73 (commit 238f42e) are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

📊 Code Coverage Summary

Type Coverage
Line 0.72
Branch

View detailed coverage report

Copy link

Build artifacts for PR #73 (commit 17297f2) are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

📊 Code Coverage Summary

Coverage Type Coverage
Overall 13.63
Changed Files 13.93

View detailed coverage report

Copy link

Build artifacts for PR #73 (commit 2a11e61) are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

Build artifacts for PR #73 (commit 23d07b0) are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

📊 Code Coverage Summary

Coverage Type Coverage
Overall 13.64
Changed Files 13.93

View detailed coverage report

Copy link

📊 Code Coverage Summary

Coverage Type Coverage
Overall 13.73
Changed Files 14.18

View detailed coverage report

Copy link

Build artifacts for PR #73 (commit 377bbe5) are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

📊 Code Coverage Summary

Coverage Type Coverage
Overall 13.71
Changed Files 14.18

View detailed coverage report

Copy link

Build artifacts for PR #73 (commit 31a1798) are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

Copy link

📊 Code Coverage Summary

Coverage Type Coverage
Overall 13.77
Changed Files 15.47

View detailed coverage report

@williscool williscool enabled auto-merge (squash) April 25, 2025 07:25
@williscool williscool mentioned this pull request Apr 25, 2025
20 tasks
@williscool williscool merged commit da17b07 into master Apr 25, 2025
10 checks passed
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