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

enhancement #3886 Internationalization in admin interface for authtoken #3887

Merged
merged 1 commit into from
Feb 1, 2016
Merged

enhancement #3886 Internationalization in admin interface for authtoken #3887

merged 1 commit into from
Feb 1, 2016

Conversation

mnacharov
Copy link
Contributor

  • verbose_name in models.Token fields
  • Meta-options verbose_name & verbose_name_plural
  • Labels in AuthTokenSerializer fields in case of usages in Brousable API
  • provide AppConfig class as described in django documentation with verbose_name came through ugettext_lazy

…ork.authtoken

+ verbose_name in models.Token fields
+ Meta-options verbose_name & verbose_name_plural
+ Labels in AuthTokenSerializer fields in case of usages in Brousable API
+ provide AppConfig class as described in django documentation with verbose_name came through ugettext_lazy
@mnacharov
Copy link
Contributor Author

I've not executed makemessages command, because didn't know is it necessary or not.

@xordoquy
Copy link
Collaborator

xordoquy commented Feb 1, 2016

It's ok, I'll run and upload the translation files.

@xordoquy xordoquy added this to the 3.3.3 Release milestone Feb 1, 2016
xordoquy added a commit that referenced this pull request Feb 1, 2016
enhancement #3886 Internationalization in admin interface for authtoken
@xordoquy xordoquy merged commit dce544e into encode:master Feb 1, 2016
@xordoquy
Copy link
Collaborator

xordoquy commented Feb 1, 2016

@mnach thanks a lot for the work !

@mnacharov mnacharov deleted the enhancement-3886 branch February 1, 2016 09:17
@itbabu
Copy link

itbabu commented Feb 26, 2016

@xordoquy Hello, is the related migration file missing?
At least django 1.9.2 tries to create a new migration file for this app after makemigrations.

@xordoquy
Copy link
Collaborator

@itbabu indeed, thanks for noticing it. I forgot that changing the model requires a migration even though the database representation doesn't change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants