Problem/Motivation

Upgrade Status was identifying as non compatible with Drupal 10

Steps to reproduce

1. A normal drupal instance of version 9.4.
2. install simple_sitemap_hidden_language module
3. Install upgrade status
4. run upgrade status check

Proposed resolution

Fix the .info.yml file

Remaining tasks

1. Review and check

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#5 simple_sitemap_hidden_language-Screenshot-after-patch.png13.31 KBarpitk
#5 simple_sitemap_hidden_language-Screenshot-before-patch.png12.29 KBarpitk
#2 d10_compatibility_simple_sitemap_hidden_language.patch520 bytesgstivanin

Issue fork simple_sitemap_hidden_language-3361000

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

gstivanin created an issue. See original summary.

gstivanin’s picture

gstivanin commented
StatusFileSize
new d10_compatibility_simple_sitemap_hidden_language.patch520 bytes

First patch generated with the proposed solution

gstivanin’s picture

gstivanin commented
Assigned: gstivanin » Unassigned
Status: Active » Needs review
Issue tags: +Drupal 10 compatibility
arpitk’s picture

arpitk commented
Assigned: Unassigned » arpitk

Reviewing this.

arpitk’s picture

arpitk commented
Assigned: arpitk » Unassigned
Issue summary: View changes
StatusFileSize
new simple_sitemap_hidden_language-Screenshot-before-patch.png12.29 KB
new simple_sitemap_hidden_language-Screenshot-after-patch.png13.31 KB

Hi @gstivanin I reviewed the patch. It applied cleanly. scanned it through upgrade status and now it D10 compatible.

Before patch

After patch

Thanks!

sophiavs made their first commit to this issue’s fork.

sophiavs opened merge request !8

murilohp’s picture

murilohp commented
Status: Needs review » Reviewed & tested by the community

Thanks for your contribution! The patches and the MR looks good to me

  • murilohp authored 4df4d678 on 2.0.x 
    Issue #3361000 by sophiavs, gstivanin, arpitk, murilohp: Drupal 10...
murilohp’s picture

murilohp commented
Status: Reviewed & tested by the community » Fixed

Commited to 2.0.x, thanks!

murilohp closed merge request !8

Status: Fixed » Closed (fixed)

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