-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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: rendering of gender field #3308
Conversation
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.
Have the server-side changes been merged?
No need for server change @mrsaicharan1 only issue was it was placed in
wrong parent if block for text type and on server it was changed to select .
…On Sun, 28 Jul, 2019, 20:18 Saicharan Reddy, ***@***.***> wrote:
***@***.**** commented on this pull request.
Have the server-side changes been merged?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#3308?email_source=notifications&email_token=AKQMTLW4EYHTIYKO6FVCDG3QBWWUHA5CNFSM4IHEGSQKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB7Y6BBY#pullrequestreview-267509895>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AKQMTLSJK6JZ2VPDTSOKQYDQBWWUHANCNFSM4IHEGSQA>
.
|
I was asking for that very change. 👍 |
@CosmicCoder96 @niranjan94 Please Review |
@uds5501 Please make sure you do |
Fixes #3307
Short description of what this resolves:
if
clause for gender fieldsScreenshot
Checklist
development
branch.