Messed up module documentation

[OrangeDog]

https://docs.saltstack.com/en/2018.3/ref/modules/all/salt.modules.service.html

There are 6 modules (group, kernelpkg, pkg, service, shadow, user) duplicated at the top of the list, and half the service modules have been renamed (launchctl, smf, systemd, upstart) so the links don't work.

[twangboy]

@jacobhammons @newwebash ^^^

[newwebash]

I'm tagging @KChandrashekhar and @Ch3LL since they're more familiar with the open Salt docs and processes than I am.

[Ch3LL]

i can definitely fix up the services issue, but I'm not quit sure what you mean by "There are 6 modules (group, kernelpkg, pkg, service, shadow, user) duplicated at the top of the list," I think you are referencing the virtual models? That should stay as we are documenting which of these modules are virtual modules

[OrangeDog]

@Ch3LL I mean they're both listed on their own at the top of the list, and again in their correct alphabetical position. If it's deliberate that they're listed separately there should be some sort of divider, and possibly no duplicate in the rest of the list.

[Ch3LL]

I believe you are referring to these virtual modules? In my opinion there is a clear division here.

[OrangeDog]

@Ch3LL see my screenshot.
There is no division in the sidebar that appears on every page.

[sagetherage]

fixed with port to master -- will be released in sodium