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

fix: DS-1240 Fix recurring event description #89

Merged
merged 1 commit into from
Dec 5, 2023

Conversation

froboy
Copy link

@froboy froboy commented Dec 5, 2023

Related Issue/Ticket:

PLEASE CHECK BASE BRANCH FOR YOUR PR
ONLY urgent and approved fixes for point release should go to master branch

https://yusa.atlassian.net/browse/DS-1240

Steps to test:

Empty description:

  • Add a Live Stream or Virtual Event with no description
  • Try to load the page

Series description:

  • Add a description to the event series
  • the description should show and the page should load

Instance description:

  • Override the description on one instance
  • the overridden description should show and the page should load

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

@froboy froboy requested review from sathyaymca and podarok December 5, 2023 20:30
@podarok podarok merged commit 91291a8 into YCloudYUSA:main Dec 5, 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