-
Notifications
You must be signed in to change notification settings - Fork 706
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
Fix#2356 Guarantor Type Selection Fixed #2373
Fix#2356 Guarantor Type Selection Fixed #2373
Conversation
Add a screen recording, screenshot wont be helpful in this case |
added |
f8fdddb
to
6ce2077
Compare
@therajanmaurya please merge my PR. |
@PratyushSingh07 please review this PR |
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.
please format your code , you can use ctrl+alt+L if you are on windows
6ce2077
to
a4e2dfd
Compare
@PratyushSingh07 I have added the changes of formatting the code. Please review it. |
fba49e6
to
5eee0ce
Compare
LGTM @therajanmaurya , we can merge this |
Fixes #2356
Removed the choicea and choiceB Options since they had no usage and were added explicitly in the code. Currently it has three Guarantor Type i.e. CUSTOMER, STAFF and EXTERNAL
Please add #hacktoberfest on merge.
VID_20231013231445.mp4