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

fix: Interface instead of instance Class #167

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

podarok
Copy link
Contributor

@podarok podarok commented Oct 28, 2021

Fixes: https://openy.slack.com/archives/C6G4ZTLVC/p1635429557024700

Related Issue/Ticket:

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

(Replace this line with a link to the related GitHub issue on ymcatwincities/openy_gated_content or your own ticketing system)

Steps to test:

  • Cron should work when s3fs module installed

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 self-assigned this Oct 28, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants