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

issue/1335 added title of the PDF schedules page #1404

Merged
merged 1 commit into from
Dec 24, 2018
Merged

issue/1335 added title of the PDF schedules page #1404

merged 1 commit into from
Dec 24, 2018

Conversation

denisbilchenko
Copy link
Contributor

Steps for review

  • Open PDF generator page (e.g. by clicking on button "Weekly PDF Schedule " on /group-exercise-classes page).
  • Make sure the title is displayed correctly(should be "Download PDF schedule").

@gundevel
Copy link
Collaborator

Build comment file:

Environment Link
Fresh OpenY installation - Rose theme http://openy.cibox.tools/build1179
Fresh OpenY installation - Lily theme http://lily.openy.cibox.tools/build1179
Upgraded(upgrade path) installation http://upgrade.openy.cibox.tools/build1179
Installation process http://install.openy.cibox.tools/build1179/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/1179/

@ddrozdik ddrozdik requested a review from podarok December 11, 2018 21:43
@ddrozdik ddrozdik added PR: Code Reviewed Code reviewed. Ready for QA PR: Needs Testing Manual testing is needed labels Dec 11, 2018
@ddrozdik ddrozdik added this to the 1.14.1 milestone Dec 11, 2018
@AndreyMaximov
Copy link
Contributor

retest this please

@gundevel
Copy link
Collaborator

Build comment file:

Environment Link
Fresh OpenY installation - Rose theme http://openy.cibox.tools/build1192
Fresh OpenY installation - Lily theme http://lily.openy.cibox.tools/build1192
Upgraded(upgrade path) installation http://upgrade.openy.cibox.tools/build1192
Installation process http://install.openy.cibox.tools/build1192/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/1192/

@gundevel
Copy link
Collaborator

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

@podarok podarok merged commit 2da14d8 into ymcatwincities:8.x-1.x Dec 24, 2018
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
PR: Code Reviewed Code reviewed. Ready for QA PR: Needs Testing Manual testing is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants