Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.A central location to track project porting status. Evolved from #2366683: Update site_map to work with 8.0.x branch.
What works
- Saving the configuration page
- Viewing the sitemap page
- Customizing the sitemap title
- Customizing the sitemap message
- #2597784: D8 port: Fix display of site map message
- #2597796: D8 port: Configure and display site's front page
- #2597808: D8 port: Configure and display site menus
- #2597804: D8 port: Configure and display site books
- #2597812: D8 port: Configure and display site vocabularies
- #2597816: D8 port: Configure and display syndication block
Direct Port/Functionality Tasks
Postponed functionality
- #2597800: D8 port: Configure and display site blogs
- #2597806: D8 port: Configure and display site FAQs
- #2605580: D8 port: Integrtate Comment RSS
- #2597788: D8 port: Configure and display RSS Legend
Cleanup Tasks
- #2599844: D8 cleanup: Convert theme functions to *.html.twig templates
- #2599848: D8 cleanup: Remove site_map_ namespacing of config parameters
Ideas for the future
| Task | Inspiration/Reference |
|---|---|
| #2613788: Clean up legacy code | |
| Check message using different text filters | #2597784-12: D8 port: Fix display of site map message |
| Additional configuration options for Books? | #2597804: D8 port: Configure and display site books |
Remove theme_site_map_menu_* functions |
#2599844: D8 cleanup: Convert theme functions to *.html.twig templates |
| #2613790: Combine sitemap.html.twig and sitemap-box.html.twig |
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | site_map_form_nitpick-2597780-9.patch | 499 bytes | akalata |
Comments
Comment #2
akalata CreditAttribution: akalata commentedComment #3
akalata CreditAttribution: akalata commentedComment #4
akalata CreditAttribution: akalata commentedAdding here for discussion - I'd like to get the module up-to-date with best practices for theming (twig templates all around, no more theme functions!) before an official release. Functionality needs to come first, though, otherwise we'll have too many moving pieces.
Comment #5
Nafes CreditAttribution: Nafes as a volunteer commentedI completely agree. Very good suggestion!
Also your roadmap and closing it piece by piece are outstanding!
Comment #6
akalata CreditAttribution: akalata commentedComment #7
Nafes CreditAttribution: Nafes as a volunteer commentedFixed #2599848: D8 cleanup: Remove site_map_ namespacing of config parameters. Name change of config parameters requires complete module uninstallation and re-installation. So, it is desirable to fix the issue early.
Also branch feature/dev-8-sync has been finished. It was kept mainly for this issue.
Comment #8
Nafes CreditAttribution: Nafes as a volunteer commentedComment #9
akalata CreditAttribution: akalata commentedAn isolated patch for a small nitpick -- the main portion of the settings form should be open by default. This is the way it is in D7 and will improve findability of the key configuration options.
Comment #10
akalata CreditAttribution: akalata commentedConsolidating notes on improvements and other ideas.
Comment #11
akalata CreditAttribution: akalata commentedComment #12
akalata CreditAttribution: akalata commentedComment #13
akalata CreditAttribution: akalata commentedComment #14
akalata CreditAttribution: akalata commentedComment #15
akalata CreditAttribution: akalata commentedComment #16
akalata CreditAttribution: akalata commentedAlmost there!
Comment #17
akalata CreditAttribution: akalata commentedComment #18
akalata CreditAttribution: akalata commentedComment #19
akalata CreditAttribution: akalata commentedComment #20
akalata CreditAttribution: akalata commentedComment #21
akalata CreditAttribution: akalata commented@Nafes, I think we're nearly there! I have a small patch in #9 that I'd love to see added.
Also wondering if there's been any discussion among the maintainers in regards to #2596861: Any chance of taking over the sitemap namespace? - if the module is going to move, that should probably get done before an alpha release, but that's not something I can really help with!
Comment #22
Nafes CreditAttribution: Nafes as a volunteer commented@akalata, great! Really there! I will take care about the patch today in the evening.
I like the idea about taking over the sitemap namespace. I was going to see main module maintainer colan soon and discuss this with him. But we are moving very fast, so I will write him today. Anyway I don't see any problem, if alpha release will be published in this project. IMHO, it is better, than to have some delay.
Comment #24
Nafes CreditAttribution: Nafes as a volunteer commentedСommitted. Thanks!
Comment #25
akalata CreditAttribution: akalata commentedComment #26
akalata CreditAttribution: akalata commentedComment #27
akalata CreditAttribution: akalata commented