- Issue created by @rsnyd
- 🇺🇸United States rsnyd
My first patch which includes some small changes identified by the upgrade_status module. Feedback is more than welcome.
- last update
over 1 year ago Patch Failed to Apply - last update
over 1 year ago Patch Failed to Apply - First commit to issue fork.
- Assigned to PrabuEla
- last update
over 1 year ago Patch Failed to Apply - Issue was unassigned.
- Status changed to Needs review
over 1 year ago 12:22pm 25 May 2023 - Assigned to PrabuEla
- last update
over 1 year ago 22 pass - 🇮🇳India PrabuEla chennai
Hi @rsnyd
Your patch was looking good, but before creating patch we need to remove the highlighted part a/docroot/modules/contrib/commerce_reports/commerce_reports.info.yml
- Issue was unassigned.
- 🇮🇱Israel jsacksick
I think the access checks should be disabled though, not enabled.
- last update
over 1 year ago Patch Failed to Apply - 🇺🇸United States rsnyd
@PrabuEla and @jsacksick, thank you both for the guidance. I've made the access checks each ->acessCheck(FALSE). New patch uploaded.
- last update
over 1 year ago 22 pass - last update
over 1 year ago 20 pass, 2 fail - 🇺🇸United States rsnyd
My apologies about the commit message format. I will fix in the next.
- last update
over 1 year ago 22 pass - @rsnyd opened merge request.
- 🇧🇷Brazil elber Brazil
Hi I revised it.
Module now is supporting drupal 10.
Changes in MR are ok.
Module is ok.
Moving to RTBC
- Status changed to RTBC
over 1 year ago 12:38pm 10 August 2023 - 🇨🇭Switzerland Lukas von Blarer
Since EOL of Drupal 9 is coming closer: Could we please get this committed?
- Status changed to Fixed
over 1 year ago 1:21pm 13 September 2023 -
jsacksick →
committed ec45f68f on 8.x-1.x authored by
rsnyd →
Issue #3362487 by rsnyd, PrabuEla, dineshkumarbollu: Drupal 10...
-
jsacksick →
committed ec45f68f on 8.x-1.x authored by
rsnyd →
Automatically closed - issue fixed for 2 weeks with no activity.