Skip to content

chore: align module naming#37

Merged
shelegdmitriy merged 1 commit into
masterfrom
chore/better-module-names
Nov 14, 2025
Merged

chore: align module naming#37
shelegdmitriy merged 1 commit into
masterfrom
chore/better-module-names

Conversation

@shelegdmitriy

Copy link
Copy Markdown
Member

This pull request makes minor adjustments to the ModuleCard component to improve how module information is displayed. The changes focus on displaying the correct module name and updating the label shown under the module title.

  • Module name and description display:
    • Changed the module name to use the label field from the manifest instead of title, falling back to module.name if label is not present.
    • Updated the subtitle under the module name to display the actual module.name instead of the static text asimov-modules.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request updates the ModuleCard component to improve module name display consistency. The changes replace the use of manifest.title with manifest.label for the display name, and replace the hardcoded "asimov-modules" text with the dynamic module.name property.

Key changes:

  • Module display name now uses manifest.label instead of manifest.title
  • Subtitle text changed from hardcoded "asimov-modules" to dynamic module.name

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/components/ModuleCard.tsx
@shelegdmitriy
shelegdmitriy force-pushed the chore/better-module-names branch from 2582d73 to bbdff4f Compare November 14, 2025 10:37
@shelegdmitriy
shelegdmitriy temporarily deployed to chore/better-module-names - asimov.directory PR #37 November 14, 2025 10:37 — with Render Destroyed
@shelegdmitriy
shelegdmitriy merged commit cffa0ff into master Nov 14, 2025
1 check passed
@shelegdmitriy
shelegdmitriy deleted the chore/better-module-names branch November 14, 2025 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants