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

GPII-3780: Solve "windows.desktopBackground" and ""windows.desktopBackgroundColor" transformations issues #754

Closed
wants to merge 11 commits into from

Conversation

JavierJF
Copy link
Contributor

@JavierJF JavierJF commented Mar 8, 2019

No description provided.

@JavierJF JavierJF changed the title GPII-3780: Solve "com.microsoft.windows.desktopBackground" transformations issues GPII-3780: Solve "windows.desktopBackground" and ""windows.desktopBackgroundColor" transformations issues Mar 8, 2019
@gpii-bot
Copy link

gpii-bot commented Mar 8, 2019

CI job passed: https://ci.gpii.net/job/universal-tests/1620/

@gpii-bot
Copy link

CI job failed: https://ci.gpii.net/job/universal-tests/1890/

@gpii-bot
Copy link

CI job failed: https://ci.gpii.net/job/universal-tests/1892/

cindyli and others added 2 commits July 26, 2019 14:23
@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/universal-tests/1893/

@@ -5102,9 +5103,23 @@
},
"capabilitiesTransformations": {
"SolidColorConfig": {
// This transform is necessary due to issue GPII-3784.
Copy link
Member

Choose a reason for hiding this comment

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

I think this transform is no longer required, right? See https://issues.gpii.net/browse/GPII-3784

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You were right, I have changed it and things are being tested in CI, I have also tested the Windows pull in local, and I will also re-trigger Windows CI once this is done.

@javihernandez
Copy link
Member

Hey @JavierJF can you update this with current master?

@JavierJF
Copy link
Contributor Author

JavierJF commented Aug 5, 2019

@javihernandez Done!

@gpii-bot
Copy link

gpii-bot commented Aug 5, 2019

CI job passed: https://ci.gpii.net/job/universal-tests/1903/

@gpii-bot
Copy link

gpii-bot commented Aug 6, 2019

CI job passed: https://ci.gpii.net/job/universal-tests/1905/

@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/universal-tests/1924/

@javihernandez
Copy link
Member

Mothballed until the upcoming work on the LSR is addressed.

@the-t-in-rtf
Copy link
Member

As discussed in the meeting, this and other pulls from JJ should be addressed once the initial work on the LSR is reviewed and approved. Yet again he is using an intra-application transform (implied setting) here, I'd rather bring this onboard once we have a better mechanism for that.

@JavierJF JavierJF closed this Oct 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants