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

Add language dropdown selector #41

Merged
merged 1 commit into from Nov 21, 2016

Conversation

andrewm89
Copy link
Member

@jirikuncar jirikuncar added this to the I3B-API-Docs milestone Nov 18, 2016
@jirikuncar jirikuncar self-assigned this Nov 18, 2016
@jirikuncar
Copy link
Member

@andrewm89 thanks! The code looks good. Can you amend commit message so it follows the inveniosoftware style or shall I amend it during squash&merge?

@andrewm89 andrewm89 force-pushed the andrewm/rfc_40/language-selector branch from 543682b to bf2faf8 Compare November 18, 2016 13:41
@andrewm89
Copy link
Member Author

@jirikuncar is it ok now?

@jirikuncar
Copy link
Member

jirikuncar commented Nov 18, 2016

@andrewm89 almost :) * Adds language dropdown macro. (addresses #40) PS: you can update also the title: jinja2: addition of language dropdown macro PSS: we might re-discuss the rules during IDF on Monday.

* Add language dropdown selector (addresses #40)

Signed-off-by: Andrew McCracken <andrew@tind.io>
@andrewm89 andrewm89 force-pushed the andrewm/rfc_40/language-selector branch from bf2faf8 to 12a8fae Compare November 18, 2016 13:45
@andrewm89
Copy link
Member Author

@jirikuncar third time lucky

@jirikuncar
Copy link
Member

LGTM

@jirikuncar jirikuncar merged commit e6c6a01 into master Nov 21, 2016
@jirikuncar jirikuncar deleted the andrewm/rfc_40/language-selector branch November 21, 2016 12:16
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