feat(reminder): add local mock adapters - #180
Merged
Merged
Conversation
Add deterministic mock adapters for reminder infrastructure and local data. Related to 1024XEngineer#177
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This was referenced Aug 10, 2026
gac0812
marked this pull request as ready for review
August 11, 2026 08:06
added 2 commits
August 11, 2026 16:10
Bring in domain and application ports from 1024XEngineer#178/1024XEngineer#179 so CI can resolve reminder modules.
Adapt mock adapters to ReminderSnoozeRequest exclusivity and ReminderConfirmedDisposition after merging application ports from main.
Avoid reusing fixed alarm/location listener handles so each registered schedule can be cancelled or unwatched independently.
Return each permission's fixed MockDeviceCapability value from requestPermission so request-then-refresh matches getStatus().
gac0812
pushed a commit
to gac0812/Tfgxs
that referenced
this pull request
Aug 11, 2026
Bring in domain, ports, and local mocks from 1024XEngineer#178/1024XEngineer#179/1024XEngineer#180 so CI can resolve composition imports.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
features/reminderbarrel,便于后续 composition 引用。Related issue
main为增量切片,单独 CI 可能因缺少 ports/domain 失败)Scope
frontend/src/shared/time/**frontend/src/infrastructure/audio/**frontend/src/infrastructure/location/**frontend/src/infrastructure/notifications/**frontend/src/features/reminder/data/local/**frontend/src/features/reminder/presentation/**frontend/src/features/reminder/index.tsOut of scope
createAppServices/AppRuntime接线(feat(reminder): wire createAppServices and AppRuntime #181)