Skip to content
This repository has been archived by the owner on Dec 13, 2022. It is now read-only.

Add more descriptive text #79

Merged
merged 14 commits into from
Apr 21, 2020
Merged

Add more descriptive text #79

merged 14 commits into from
Apr 21, 2020

Conversation

aguilerapy
Copy link
Contributor

#47

@aguilerapy aguilerapy requested review from romifz and removed request for romifz February 25, 2020 11:50
Copy link
Contributor

@romifz romifz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is good, I just flagged a couple of things to fix.

default/templates/default/index.html Outdated Show resolved Hide resolved
locale/es/LC_MESSAGES/django.po Outdated Show resolved Hide resolved
locale/es/LC_MESSAGES/django.po Outdated Show resolved Hide resolved
# Conflicts:
#	locale/es/LC_MESSAGES/django.po
@aguilerapy aguilerapy requested a review from romifz March 9, 2020 12:18
# Conflicts:
#	locale/es/LC_MESSAGES/django.po
@aguilerapy aguilerapy requested a review from romifz March 23, 2020 17:22
default/templates/default/index.html Outdated Show resolved Hide resolved
default/templates/default/index.html Outdated Show resolved Hide resolved
@aguilerapy aguilerapy requested a review from romifz March 26, 2020 18:40
Copy link
Contributor

@romifz romifz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hola Andrés, te faltó quitar las etiquetas <li> de las traducciones, aparte marqué otras cosas pequeñas.

@aguilerapy aguilerapy requested a review from romifz March 30, 2020 20:36
@aguilerapy
Copy link
Contributor Author

Can you take a look and merge if it's correct? @jpmckinney

@jpmckinney
Copy link
Member

I haven't read carefully, but we can always tweak the text later :)

@jpmckinney jpmckinney merged commit 75e2574 into master Apr 21, 2020
@jpmckinney jpmckinney deleted the 47-add_text branch April 21, 2020 20:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants