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

feat(chromecast)!: Fork receiver app v1 #94

Merged
merged 1 commit into from
May 17, 2024

Conversation

joeyparrish
Copy link
Member

To prepare for a v2 client and receiver app, this copies receiver.html (with some additional comments) to receiver-v1.html. After merging this change, the registration for the v1 receiver app ID will be changed to point to receiver-v1.html. This will free us to make breaking changes in receiver.html under a new app ID.

It is important for a smooth transition to make a copy of receiver.html first and not simply rename it, because there will be a transition period where some Chromecasts under test will still refer to the old URL. This should be < 48 hours after merge.

To prepare for a v2 client and receiver app, this copies receiver.html (with some additional comments) to receiver-v1.html.  After merging this change, the registration for the v1 receiver app ID will be changed to point to receiver-v1.html.  This will free us to make breaking changes in receiver.html under a new app ID.
@joeyparrish joeyparrish changed the title feat!(chromecast): Fork receiver app v1 feat(chromecast)!: Fork receiver app v1 May 17, 2024
Copy link

@alekzz alekzz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit; Is it possible to mark file as copied to preserve the history?

@joeyparrish
Copy link
Member Author

No, git will always see it as a new file. However, the overall history is still preserved. You can see the development of v1 over time in receiver.html's history.

@joeyparrish joeyparrish merged commit a6a7998 into shaka-project:main May 17, 2024
11 of 16 checks passed
@joeyparrish joeyparrish deleted the prepare-for-v2 branch May 17, 2024 23:24
@github-actions github-actions bot added the status: archived Archived and locked; will not be updated label Jul 16, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: archived Archived and locked; will not be updated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants