- Merge request !2688Issue #3261452: [PP-1] Remove tracker module from core โ (Closed) created by spokje
- ๐ฌ๐งUnited Kingdom catch
I think this is at the point where it can land once the 11.0.x branch is actually open.
- ๐ณ๐ฟNew Zealand quietone
Changing the parent to the removal meta.
- ๐ญ๐บHungary Gรกbor Hojtsy Hungary
Updating title, tags and version number based on recent announcement at https://www.drupal.org/about/core/blog/new-drupal-core-branching-scheme-... โ
- ๐ฉ๐ฐDenmark ressa Copenhagen
Thanks for cleaning up, and keeping Drupal lean. I found this issue while looking for a solution to a Tracker-related error during a Drupal 7 to Drupal 10 migration of Views:
Source ID tracker: VIEW: tracker DISPLAY: default - The 'uid_touch' argument Plugin is missing and will be listed as broken.
Source ID tracker: VIEW: tracker DISPLAY: default - The field timestamp does not exist and has been removed from the fields configuration.The solution was to delete the Contextual filters "User posted or commented" field from the Tracker view (it could not be deleted).
- ๐ณ๐ฑNetherlands spokje
Spokje โ changed the visibility of the branch 3261452-remove-tracker-module to hidden.
- ๐ณ๐ฑNetherlands spokje
Spokje โ changed the visibility of the branch drupal-3261452 to hidden.
- ๐ณ๐ฑNetherlands spokje
Spokje โ changed the visibility of the branch 11.x to hidden.
- Status changed to Needs work
7 months ago 2:33am 7 April 2024 - ๐ซ๐ทFrance andypost
old MR needs rebase, I opened new to check other aproach instead of clean-up of dumps
- Status changed to Needs review
7 months ago 5:03am 7 April 2024 - ๐ซ๐ทFrance andypost
Got tests green with fixtures clean-up https://git.drupalcode.org/project/drupal/-/merge_requests/7358 (split into 3 commits)
Not possible to use obsolete stub here as there's configuration
tracker.settings
in the filled dump, so probably it should wait for fixtures update - Status changed to Needs work
7 months ago 12:02pm 7 April 2024 The Needs Review Queue Bot โ tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily 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
7 months ago 8:04pm 19 April 2024 - ๐ฌ๐งUnited Kingdom catch
Overall looks good, but a couple of questions on the MR.
- ๐บ๐ธUnited States smustgrave
smustgrave โ changed the visibility of the branch 3261452-11.x-remove-tracker to hidden.
- Status changed to RTBC
7 months ago 9:35pm 19 April 2024 - ๐บ๐ธUnited States smustgrave
Took a look at the two comments but they appear to already be in getMissingPaths() seems correct based on d6 migration tests.
Rests looks good.
- Status changed to Fixed
7 months ago 9:46pm 19 April 2024 - ๐ฌ๐งUnited Kingdom catch
Ahh thanks for checking properly, just the @todo was wrong then.
Committed/pushed to 11.x, thanks!
Automatically closed - issue fixed for 2 weeks with no activity.