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

Contains fix for bug #79 #113

Closed
wants to merge 3 commits into from
Closed

Conversation

kspviswa
Copy link
Contributor

@kspviswa kspviswa commented Aug 6, 2015

It is quite a bug :) . The home country span element was missing the create.html.template file. I just added the same (copying from view.html.template). Also used the "trans" element to support language compatibility.

Unit tested & found OK. Attached screen shots.

Home country field now visible

image

image

Viewing the added record with the added Home country

image

@gimite
Copy link
Contributor

gimite commented Aug 7, 2015

Thanks for the fix! I will take a look.

@gimite
Copy link
Contributor

gimite commented Aug 10, 2015

The change looks good to me. Before pulling your change, can you sign this agreement:
https://cla.developers.google.com/about/google-individual
This is required to accept patches from people outside our company, per our policy.

The CLA is necessary mainly because you own the
copyright to your changes, even after your contribution becomes part of our
codebase, so we need your permission to use and distribute your code.

@kspviswa
Copy link
Contributor Author

@gimite

Thanks for accepting my pull request. As directed, I had signed the CLA under individual category & tagged my github account (kspviswa) to it. Please let me know, if anything else is pending from my side.

BR,
Viswa

@gimite
Copy link
Contributor

gimite commented Aug 12, 2015

Cherry-picked c70b55e . Thanks!

@gimite gimite closed this Aug 12, 2015
@kspviswa kspviswa deleted the kspviswa-bugfix_#79 branch August 15, 2015 05:18
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.

None yet

2 participants