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

Spanish Translations Fix #336

Closed
wants to merge 48 commits into from
Closed

Spanish Translations Fix #336

wants to merge 48 commits into from

Conversation

ibarrajo
Copy link
Contributor

update some lines in the spanish translation

@benedmunds
Copy link
Owner

Hey dude, you have a lot of extra files in the this PR.

@benedmunds benedmunds closed this Oct 22, 2012
@ibarrajo
Copy link
Contributor Author

sorry, something got messed up while I was comitting

I only edited the spanish translation file

On Mon, Oct 22, 2012 at 12:07 PM, Ben Edmunds [email protected]:

Hey dude, you have a lot of extra files in the this PR.


Reply to this email directly or view it on GitHubhttps://github.com//pull/336#issuecomment-9676378.

@benedmunds
Copy link
Owner

Can you send a new PR with just the spanish file changes?

@ibarrajo
Copy link
Contributor Author

sure =]

On Tue, Oct 23, 2012 at 10:46 AM, Ben Edmunds [email protected]:

Can you send a new PR with just the spanish file changes?


Reply to this email directly or view it on GitHubhttps://github.com//pull/336#issuecomment-9711100.

@ibarrajo
Copy link
Contributor Author

what branch needs the update?

I see everything is translated in the spanish file, I haven't used github
before,
so I'm afraid to mess something up again.

On Tue, Oct 23, 2012 at 10:53 AM, Josue Ibarra [email protected]:

sure =]

On Tue, Oct 23, 2012 at 10:46 AM, Ben Edmunds [email protected]:

Can you send a new PR with just the spanish file changes?


Reply to this email directly or view it on GitHubhttps://github.com//pull/336#issuecomment-9711100.

@benedmunds
Copy link
Owner

I went ahead and updated this manually. Thanks!

Dentxinho added a commit to Dentxinho/CodeIgniter-Ion-Auth that referenced this pull request Jan 4, 2013
# By Ben Edmunds (9) and others
# Via Ben Edmunds (14) and Vytenis (1)
* upstream/2: (24 commits)
  Update language/lithuanian/ion_auth_lang.php
  Add ability to change groups for a user.
  Added conditional loading of the session lib/driver based on CI2 or 3 version
  - Forgot password label based on identity
  fix benedmunds#368
  Optimized create_group existing group check
  Added CI3 compatible session driver loading
  Added sess_create back to logout for CI 2
  Removed sess_create() from logout() method for CI3.0 compatibility
  Fixed reference to 'groups' table
  Some naming changes, and moved new password hashing into where it is needed.
  Removed unused $db_password from ion_auth_model::change_password().
  Some improvements in Persian translation
  Updated misguided docs for users()
  Added check for use_ci_email before loading email config.  This resolves issue benedmunds#352.
  Persian Translation
  Bug Fix, and Enhancements
  Updated spanish translation (via ibarrajo).  Closes PR benedmunds#336
  Added romanian translation (via Adrian Voicu)
  forgotten password did not work if identity was anything other than email. now it does.
  ...
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.

2 participants