Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

[CR-42] Fix dependency to gallery config (crop.type.crop_1920_700) #2137

Merged
merged 1 commit into from Sep 3, 2020

Conversation

hamrant
Copy link
Contributor

@hamrant hamrant commented Sep 2, 2020

Original Issue, this PR is going to fix:

This PR brokes OpenY fresh install, because there exist dependency to crop.type.crop_1920_700 config from openy_prgf_gallery

Steps for review

  • Check that OpenY fresh install works fine (test on Open Y Installation Wizard build)

@hamrant hamrant added PR: Needs Review Needs someone review ( code ) PR: Needs Testing Manual testing is needed labels Sep 2, 2020
@hamrant hamrant requested a review from podarok September 2, 2020 17:53
@hamrant hamrant self-assigned this Sep 2, 2020
@gundevel
Copy link
Collaborator

gundevel commented Sep 2, 2020

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build2499/install.php


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/2499/

@gundevel
Copy link
Collaborator

gundevel commented Sep 2, 2020

Acessibility Sniffer: front page checking WCAG2AA http://openy.cibox.tools/build2499/frontWCAG2AAhtmlcs.txt
Acessibility Sniffer: join page checking WCAG2AA http://openy.cibox.tools/build2499/joinWCAG2AAhtmlcs.txt
Acessibility Sniffer: locations page checking WCAG2AA http://openy.cibox.tools/build2499/locationsWCAG2AAhtmlcs.txt
Acessibility Sniffer: schedules page checking WCAG2AA http://openy.cibox.tools/build2499/schedulesWCAG2AAhtmlcs.txt
Acessibility Sniffer: blog page checking WCAG2AA http://openy.cibox.tools/build2499/blogWCAG2AAhtmlcs.txt

@gundevel
Copy link
Collaborator

gundevel commented Sep 2, 2020

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build2499/

@podarok podarok merged commit e344153 into ymcatwincities:8.x-2.x Sep 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
PR: Needs Review Needs someone review ( code ) PR: Needs Testing Manual testing is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants