-
Notifications
You must be signed in to change notification settings - Fork 32
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
Prop 'type' doesn't change keyboard type #68
base: main
Are you sure you want to change the base?
Conversation
@BogdanGorelkin thank you, I'll test and merge |
bumping this |
src/OtpInput/OtpInput.tsx
Outdated
@@ -5,7 +5,7 @@ import { OtpInputProps, OtpInputRef } from "./OtpInput.types"; | |||
import { VerticalStick } from "./VerticalStick"; | |||
import { useOtpInput } from "./useOtpInput"; | |||
|
|||
export const OtpInput = forwardRef<OtpInputRef, OtpInputProps>((props, ref) => { | |||
export const OtpInput = forwardRef<OtpInputRef, OtpInputProps>((props: OtpInputProps, ref) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need for this change; props types getting populated from forwardRef<>
generics
@@ -15,7 +15,8 @@ | |||
"test:coverage": "jest --collectCoverage --coverageDirectory=\"./coverage\"", | |||
"build:dev": "tsc", | |||
"build:prod": "tsc -p tsconfig.prod.json", | |||
"prepublishOnly": "npm install && npm run build:prod" | |||
"prepublishOnly": "npm install && npm run build:prod", | |||
"prepare": "npm run build:prod" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do we need it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Without prepare script I was not able to get dist folder while installing my fork as:
npm install git@github.com:BogdanGorelkin/react-native-otp-entry.git
I tested, works fine, thank you. Just update the snapshot and address my comments, then we'll be good to go :) |
@BogdanGorelkin hey, any updates here? |
Hi, @anday013 , sure, check the answers above to your comments. |
can we merge this? |
When prop type was set to 'alphanumeric' the keyboard remains numeric.
Please approve it, because now it works perfectly for both types.