I have checked module via drupal-check and found below issues

Line src/Form/VideoForm.php
------ -------------------------------------------------------------------
87 Call to deprecated method l() of class Drupal\Core\Form\FormBase:
in Drupal 8.0.0 and will be removed before Drupal 9.0.0.
Use \Drupal\Core\Link instead.
------ -------------------------------------------------------------------

[ERROR] Found 1 error

CommentFileSizeAuthor
#2 domain_video_sitemap-3076974.patch927 bytessushilpal
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sushilpal created an issue. See original summary.

sushilpal’s picture

sushilpal CreditAttribution: sushilpal commented
FileSize
domain_video_sitemap-3076974.patch927 bytes

patch for this.

sushilpal’s picture

sushilpal CreditAttribution: sushilpal commented
Status: Active » Needs review
i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
CreditAttribution: i-trokhanenko at RockSolid PSF, Drupal Ukraine Community commented
Assigned: sushilpal » Unassigned
i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
CreditAttribution: i-trokhanenko at RockSolid PSF, Drupal Ukraine Community commented
Version: 8.x-1.0 » 8.x-1.x-dev
Status: Needs review » Reviewed & tested by the community

Hi @sushilpal
Patch #2 works well for me. +1 RTBC
Thanks!

Andrey Proh’s picture

Andrey Proh CreditAttribution: Andrey Proh at RockSolid PSF commented

+1 RTBC

i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
CreditAttribution: i-trokhanenko at RockSolid PSF, Drupal Ukraine Community commented
i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
CreditAttribution: i-trokhanenko at RockSolid PSF, Drupal Ukraine Community commented
Issue tags: +RockSolidCodeSprint
i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
CreditAttribution: i-trokhanenko at RockSolid PSF, Drupal Ukraine Community commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.