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

remove everything #1804

Merged
merged 2 commits into from Sep 15, 2020
Merged

remove everything #1804

merged 2 commits into from Sep 15, 2020

Conversation

adrien-de-tocqueville
Copy link
Contributor

Purpose of this PR

This PR removes the CloudLayer because it needs to be reworked.
Disables the appropriate test and remove related docs.

Waiting for yamato to work again

Automated Tests: What did you setup? (Add a screenshot or the reference image of the test please)

Yamato: (Select your branch):
https://yamato.prd.cds.internal.unity3d.com/jobs/902-Graphics/tree/HDRP%252Fdisable-cloud-layer

@@ -208,6 +208,8 @@ void UpdateSkyIntPopupData()
VolumeProfile profile = m_SerializedActiveSceneLightingSky.volumeProfile;
if (profile != null)
{
bool foundID = false;
Copy link
Collaborator

Choose a reason for hiding this comment

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

What is this change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because when testing, I noticed that if you select a sky for the static lighting and then you delete it, the UI shows a blank dropdown, so it's just a small fix

@sebastienlagarde sebastienlagarde marked this pull request as ready for review September 15, 2020 18:57
@sebastienlagarde sebastienlagarde merged commit 1b34be9 into HDRP/staging Sep 15, 2020
@sebastienlagarde sebastienlagarde deleted the HDRP/disable-cloud-layer branch September 15, 2020 18:58
@sebastienlagarde sebastienlagarde mentioned this pull request Sep 17, 2020
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants