Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve the skill form #145

Merged
merged 1 commit into from
Sep 4, 2024
Merged

Conversation

vishnoianil
Copy link
Member

. Skill form is updated for the similar look and feel as knowledge form.
. Mandatory seed exampels are not allowed to delete, but additional seed exampel added by user can be deleted. . Each input field has a validation check now.
. Edit of skill submission uses the same form as main skill form. . Refactored skill form similar to knowledge form. . Enforced all the constrained imposed by skill schema.

Fixes #117

@vishnoianil vishnoianil force-pushed the overhaul-skills branch 2 times, most recently from b67f06d to 69a8218 Compare September 4, 2024 15:06
*.* Skill form is updated for the similar look and feel
as knowledge form.
*.* Mandatory seed exampels are not allowed to delete, but
additional seed exampel added by user can be deleted.
*.* Each input field has a validation check now.
*.* Edit of skill submission uses the same form as main skill form.
*.* Refactored skill form similar to knowledge form.
*.* Enforced all the constrained imposed by skill schema.

Signed-off-by: Anil Vishnoi <[email protected]>
@nerdalert nerdalert merged commit eda9e97 into instructlab:main Sep 4, 2024
5 checks passed
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.

Improve UX of existing Knowledge contribution form
2 participants