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

[DS-762] Fix the Virtual Y Content block for the Layout Builder #36

Merged
merged 1 commit into from
Jun 9, 2023

Conversation

aleevas
Copy link

@aleevas aleevas commented Jun 8, 2023

Related Issue/Ticket:
DS-762

This fix also should resolve this issue in DBlog

DS-762png

Steps to test:

  • Enable Layout Builder module.
  • Enable openy_gc_auth_example module.
  • Create Content Type for Layout Builder.
  • Create node and add Virtual Y Content Block via Layout Builder.
  • Create another node with Virtual Y Login Block via Layout Builder.
  • Set up Virtual Y Landing Page url and Virtual Y Login Landing Page url fields on settings page: admin/openy/virtual-ymca
  • Navigate to the page with Virtual Y Content Login block and hit Enter Virtual Y button.
  • Observe redirect to the node with Virtual Y Content Block

Quality checks:

Please check these boxes to confirm this PR covers the following cases:

  • Maintaining our upgrade path is essential. Check one or the other:
    • This PR provides updates via hook_update_N or other means.
    • No updates are necessary for this change.
  • Front end fixes should be tested against all of the Open Y Themes.
    • Tested against Carnation
    • Tested against Lily
    • Tested against Rose
    • This change does not contain front-end fixes.
  • I have flagged this PR "Needs Review" or pinged the VY devs/QA
    team in Slack

@podarok podarok merged commit fffbc2a into YCloudYUSA:master Jun 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants