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

[MAINTAIN-116] Fix blog listings on Carnation #2505

Merged
merged 2 commits into from
Jul 1, 2021

Conversation

hamrant
Copy link
Contributor

@hamrant hamrant commented Jul 1, 2021

Original Issue, this PR is going to fix: https://openy.atlassian.net/browse/MAINTAIN-116

Steps for review

  • Install extended or custom Open Y version with Carnation theme
  • Go to /blog/category/community-outreach
  • Check that blog teaser fixed and displayed in the columns

Screenshot from 2021-07-01 17-10-44

  • Compare with https://sandbox-carnation-cus.openy.org/blog/category/community-outreach
  • Check that fix works only for blog taxonomy (the rest of the views should look the same)
    • Check any item from Amenities list /admin/structure/taxonomy/manage/amenities/overview
    • Check any item from Area list /admin/structure/taxonomy/manage/area/overview
    • Check any item from News Category list /admin/structure/taxonomy/manage/news_category/overview
  • Non Profit

@hamrant hamrant added PR: Needs Review Needs someone review ( code ) PR: Needs Testing Manual testing is needed Five Jars Assigned for resolution labels Jul 1, 2021
@hamrant hamrant self-assigned this Jul 1, 2021
@gundevel
Copy link
Collaborator

gundevel commented Jul 1, 2021

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3263/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/3263/

@gundevel
Copy link
Collaborator

gundevel commented Jul 1, 2021

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

@hamrant hamrant requested a review from dchubar-fj July 1, 2021 14:27
@gundevel
Copy link
Collaborator

gundevel commented Jul 1, 2021

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3264/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/3264/

@podarok
Copy link
Contributor

podarok commented Jul 1, 2021

I changed this with your permission )))
image
Looks great
Thank you @hamrant

@gundevel
Copy link
Collaborator

gundevel commented Jul 1, 2021

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

@hamrant
Copy link
Contributor Author

hamrant commented Jul 1, 2021

@podarok thanks! 😂

@podarok podarok merged commit 4f31587 into ymcatwincities:9.x-2.x Jul 1, 2021
@podarok podarok added PR: Code Reviewed Code reviewed. Ready for QA PR: Tested Manually tested. Green light for merging Type: Bug Bug. Something is not working as intended. and removed PR: Needs Review Needs someone review ( code ) PR: Needs Testing Manual testing is needed labels Jul 1, 2021
@hamrant hamrant deleted the MAINTAIN-116 branch July 1, 2021 14:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Five Jars Assigned for resolution PR: Code Reviewed Code reviewed. Ready for QA PR: Tested Manually tested. Green light for merging Type: Bug Bug. Something is not working as intended.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants