Skip to content

refactored the skill description#121

Merged
aevo98765 merged 2 commits intoinstructlab:mainfrom
MuhammadKurdi-CS:issue/#114/refactor-skill-description
Sep 2, 2024
Merged

refactored the skill description#121
aevo98765 merged 2 commits intoinstructlab:mainfrom
MuhammadKurdi-CS:issue/#114/refactor-skill-description

Conversation

@MuhammadKurdi-CS
Copy link
Collaborator

refactored skill description

Signed-off-by: Muhammad Mohammadi <muhammadmohammadis6@gmail.com>
import YamlCodeModal from '../../YamlCodeModal';
import { AttributionData, SchemaVersion, SkillYamlData } from '@/types';
import SkillDescription from './SkillDescription';
import SkillDescription from './SkillDescription/SkillDescriptionContent';
Copy link
Member

Choose a reason for hiding this comment

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

This is not the correct component to import. You have imported just the content. The 'wrapper' component SkillDescription is what should be imported here.

If you run npm run dev you can see that the Expandable section is not being rendered.

Compare the your branches UI to the main branch UI and this should make sense.

Copy link
Collaborator Author

@MuhammadKurdi-CS MuhammadKurdi-CS Aug 23, 2024

Choose a reason for hiding this comment

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

thank you for bringing this to my attention, the rendering issue has been resolved.

Signed-off-by: Muhammad Mohammadi <muhammadmohammadis6@gmail.com>
Copy link
Member

@vishnoianil vishnoianil left a comment

Choose a reason for hiding this comment

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

LGTM

@aevo98765 aevo98765 merged commit 2e3b585 into instructlab:main Sep 2, 2024
@aevo98765
Copy link
Member

Thanks for this one @MuhammadKurdi-CS

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.

3 participants