Add core_version_requirement: ^8 || ^9 to video_embed_facebook.info.yml to designate that the module is compatible with Drupal 9. See https://www.drupal.org/node/3070687.

CommentFileSizeAuthor
#2 3131703-2.patch404 bytesyogeshmpawar

Comments

yogeshmpawar created an issue. See original summary.

yogeshmpawar’s picture

Assigned: yogeshmpawar » Unassigned
Status: Active » Needs review
Issue tags: +Drupal 9 compatibility, +Drupal 9 readiness, +Drupal 9 porting day
StatusFileSize
new404 bytes

Here's the patch for Drupal 9 readiness.

kristen pol’s picture

Title: Drupal 9 readiness » Drupal 9 compatibility
Issue tags: -Drupal 9 readiness

Per a Slack discussion with Gábor Hojtsy last year regarding usage of D9 tags (Drupal 9, Drupal 9 compatibility, Drupal 9 readiness, etc.), "Drupal 9 compatibility" should be used for contributed projects that need updating and "Drupal 9" was the old tag for D8 issues before the D9 branch was ready. Doing title and tag cleanup here based on that discussion. Please use "Drupal 9 compatibility" for titles and tags on other issues. Thanks.

smrutha’s picture

Status: Needs review » Reviewed & tested by the community

Patch #2 tested on Drupal 9. Works good.

sam152’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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