- Issue created by @quietone
- First commit to issue fork.
- Status changed to Needs review
about 1 year ago 4:20am 2 December 2023 - πΊπΈUnited States smustgrave
core/modules/config/tests/src/Functional/ConfigInstallProfileOverrideTest.php: // language tour has a dependency on this tour so it has to exist. core/modules/config/tests/src/Functional/ConfigInstallProfileOverrideTest.php: // Ensure the authenticated role has the access tour permission.
Is covered in π Decouple tour from ConfigInstallProfileOverrideTest Needs review .
- Status changed to Needs work
about 1 year ago 4:56am 2 December 2023 - Status changed to Needs review
about 1 year ago 3:22pm 2 December 2023 - πΊπΈUnited States smustgrave
Addressed the feedback, added a note in the tour.api.php, I'm sure it doesn't follow standard but serves as a reminder for when we merge tour back into core.
- Status changed to Needs work
about 1 year ago 3:47pm 2 December 2023 The Needs Review Queue Bot β tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
This does not mean that the patch needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide β to find step-by-step guides for working with issues.
- Status changed to Needs review
about 1 year ago 4:05pm 2 December 2023 - Status changed to Needs work
about 1 year ago 8:05pm 8 December 2023 - π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
Left a comment/suggestion on the MR
- Status changed to Needs review
about 1 year ago 5:52pm 11 December 2023 - πΊπΈUnited States smustgrave
Removed change to tour.api and added to TourViewBuilder.
- Status changed to RTBC
about 1 year ago 11:13pm 13 December 2023 - π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
Looks good to me thanks
- Status changed to Fixed
12 months ago 8:11am 21 December 2023 Automatically closed - issue fixed for 2 weeks with no activity.