-
Notifications
You must be signed in to change notification settings - Fork 4
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
Drupal 9 compatibility fixes. #8
base: 9.x-2.x
Are you sure you want to change the base?
Conversation
c040d3c
to
553dc9d
Compare
553dc9d
to
38cfd21
Compare
38cfd21
to
12d3c53
Compare
@NightHunterSV I've tested pieces of this on a site and while there's a lot i haven't tested, there's one change necessary. digital_signage/modules/openy_digital_signage_playlist/src/Form/OpenYPlaylistItemForm.php Line 132 in 12d3c53
should be
Otherwise the Edit/Add form for playlist items at |
Also digital_signage/modules/openy_digital_signage_playlist/openy_digital_signage_playlist.module Line 121 in 12d3c53
Call to a member function addClass() on array ...
so just above it (line 120) should add:
|
Fix playlist's Duration field validation
@froboy thanks for your feedback. Those issues have been addressed, the fixes are incorporated in this branch. |
No description provided.