-
Notifications
You must be signed in to change notification settings - Fork 706
Port Main Window to React #4853
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
base: master
Are you sure you want to change the base?
Conversation
BundleMonFiles updated (2)
Unchanged files (2)
Total files change -15.21KB -0.12% Final result: ✅ View report in BundleMon website ➡️ |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
)} | ||
</div> | ||
{(!ctrl.platform || | ||
(isPlatform(['youtube', 'facebook', 'twitter', 'tiktok', 'kick']) && !isStreaming)) && ( |
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.
not really a comment, but a note to self, lots of isPlatform
evaluations over here, most of the time with the same arguments, that should probably be extracted in the future. It will make it easier to support newer platforms as well instead of having to track all these instances
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.
i think that's slightly out of scope for this PR but yes agreed, i have a thought to handle this similar how we use hasCapability
for platform features
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.
made a follow up PR here for when this is merged
#5401
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
No description provided.