Skip to content

Conversation

@matebarabas
Copy link
Contributor

@matebarabas matebarabas commented Sep 9, 2025

Overview/Summary

Eliminate unnecessary initialization of hasListableModules in the shortcode and enhance the child module status display by adding icons for better clarity.

This PR fixes/adds/changes/removes

  1. Fix proposed module index tables by removing unnecessary initialization of hasListableModules
image 2. Add child module status to the tooltip in module index tables to explicitly show when a child module is not available yet, despite the parent being available. image

Breaking Changes

n/a

As part of this Pull Request I have

  • Read the Contribution Guide and ensured this PR is compliant with the guide
  • Checked for duplicate Pull Requests
  • Associated it with relevant GitHub Issues or ADO Work Items (Internal Only)
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Ensured PR tests are passing
  • Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.)

@matebarabas matebarabas requested a review from a team as a code owner September 9, 2025 04:21
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs: Triage 🔍 Maintainers need to triage still label Sep 9, 2025
@microsoft-github-policy-service

Important

The "Needs: Triage 🔍" label must be removed once the triage process is complete!

Tip

For additional guidance on how to triage this issue/PR, see the AVM Issue Triage documentation.

@matebarabas matebarabas self-assigned this Sep 9, 2025
@matebarabas matebarabas added Type: Documentation 📄 Improvements or additions to documentation and removed Needs: Triage 🔍 Maintainers need to triage still labels Sep 9, 2025
@jeanchg jeanchg merged commit 7ddf36e into main Sep 9, 2025
7 checks passed
@jeanchg jeanchg deleted the feat/matebarabas/site-script-fix branch September 9, 2025 06:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Documentation 📄 Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants