Fix Prime Video platform dedupe#9
Conversation
120318
force-pushed
the
fix/prime-platform-dedupe
branch
2 times, most recently
from
12bc695 to
f9cb4ea
Compare
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f9cb4ea227
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
120318
force-pushed
the
fix/prime-platform-dedupe
branch
from
f9cb4ea to
4dbdc02
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4dbdc02d28
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
120318
left a comment
120318
left a comment
There was a problem hiding this comment.
Follow-up pushed in 1504be2.
This removes source-specific networks and online_platforms from MediaFullInfo. The aggregate now exposes schedule platform data through schedule.platforms only, while profile/scope storage keeps its source-specific fields internally. Scope/profile projection now derives network roles from TV airings and online roles from schedule platforms without reintroducing those source fields on the public aggregate.
Validation:
./aethera.sh test-backend tests/test_media_context_resolution.py tests/test_media_profile_service_simple_info.py tests/test_media_detail_overview.py tests/test_media_provider_service_tmdb_source.py tests/test_media_provider_service_tmdb_override.py tests/test_media_source_canonical.py./scripts/check_quality.sh
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1504be2276
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 914dfc58f7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ba2832467e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex 看看 |
|
Summary
Testing
|
|
Codex Review: Didn't find any major issues. 🚀 ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
1 participant
Summary
MediaScheduleSummary.platformsthe single schedule platform contract exposed across services and APIs.networksandonline_platformson media/profile storage models only, not on schedule summaries or overview schedule consumers.schedule.platforms.Testing
./aethera.sh test-backend tests/test_media_context_resolution.py./aethera.sh test-backend tests/test_media_detail_overview.py tests/test_media_context_resolution.py./scripts/docker_compose.sh run --rm --no-deps frontend npm run quality./scripts/check_quality.shNotes
summary.networks,summary.online_platforms,schedule.networks, orschedule.online_platformsconsumers in backend service/frontend schedule paths.