-
Notifications
You must be signed in to change notification settings - Fork 28
macos duckplayer #459
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
macos duckplayer #459
Conversation
Current dependencies on/for this PR: This comment was auto-generated by Graphite. |
b0ad2bb
to
d00eebd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor changes needed :)
Thanks
Sorry should have marked as draft, not quite ready for review - although the points raised are still valid, thanks :) |
bab287d
to
a2933e8
Compare
14c4132
to
0fdd364
Compare
5e651c4
to
03e0623
Compare
c71facd
to
02c4f37
Compare
b3c4fdb
to
c642191
Compare
c642191
to
1acfdf2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few comments and then should be good to go
@jonathanKingston - should be everything addressed now - I'll create a follow-up task in asana to review messaging construction once both mine + @franfaccin 's PRs have landed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@jonathanKingston time to get this reviewed/merged
apple-isolated
(Duck Player Overlays being the only feature in it)