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

[Typescript] Bug fix to allow custom components #875

Closed
wants to merge 1 commit into from

Conversation

bala-one
Copy link

@bala-one bala-one commented Dec 8, 2021

Description / Motivation

Support for custom components in field-factory.tsx I am trying to add a custom react component to the jss-react-forms and relying on typescript.
The method definition only allows of type FieldTypes Enum and I dont have an override to pass my custom component implementation. Internally its a map that takes string and react component for that type.

I have added string so it can be FieldTypes for OOB field and custom field can be added with the {guid} string.

Testing Details

Sitecore 10.1 JSS SXA Docker environment Typescript
In my local instance I am using this approach and it works.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Support for custom components in field-factory.tsx I am trying to add a custom react component to the jss-react-forms and relying on typescript.
The method definition only allows of type FieldTypes Enum and I dont have an override to pass my custom component implementation. Interally its a map that takes string and react component for that type.

I have added string so it can be FieldTypes for OOB field and custom field can be added with the {guid} string.
@vercel
Copy link

vercel bot commented Dec 8, 2021

@bala-one is attempting to deploy a commit to the Sitecore JSS Team on Vercel.

A member of the Team first needs to authorize it.

@bala-one
Copy link
Author

bala-one commented Dec 8, 2021

@CobyPear The new pull request with proper branch.

@CobyPear CobyPear requested a review from a team December 8, 2021 21:11
@CobyPear
Copy link
Contributor

CobyPear commented Dec 9, 2021

Our CI shows a linting error, can you run yarn lint on the repo and address any errors please? Thanks.

@illiakovalenko
Copy link
Contributor

illiakovalenko commented May 31, 2022

@bala-one Merged into #1047
so I close current PR

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