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 button to admin ui #1738

Merged
merged 4 commits into from
May 19, 2017
Merged

Conversation

DblK
Copy link
Member

@DblK DblK commented May 17, 2017

This PR is only for adding a "Create New Organization" in the admin UI panel (see screenshot bellow).

Before:
image

After:
image

@lunny lunny added type/enhancement An improvement of existing functionality topic/ui Change the appearance of the Gitea UI labels May 17, 2017
@lunny lunny added this to the 1.3.0 milestone May 17, 2017
@@ -1134,6 +1134,7 @@ orgs.org_manage_panel = Organization Manage Panel
orgs.name = Name
orgs.teams = Teams
orgs.members = Members
orgs.new_orga = Create New Organization
Copy link
Contributor

Choose a reason for hiding this comment

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

It should be "Create Organization". Create implies the word new.

Copy link
Member Author

Choose a reason for hiding this comment

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

Okay will update the ini file in next commit

@@ -309,7 +309,7 @@ password_change_disabled=Les utilisateurs non-locaux n'ont pas le droit de modif
emails=Adresses e-mail
manage_emails=Gérer les adresses e-mail
email_desc=Votre adresse e-mail principale sera utilisée pour les notifications et d'autres opérations.
primary=Principale
primary=Principalef
Copy link
Member

@sapk sapk May 18, 2017

Choose a reason for hiding this comment

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

Probably a typo.
Futhermore, only en-US should be updated the rest will be updated via crowdin.com

Copy link
Member

Choose a reason for hiding this comment

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

^

Copy link
Member Author

Choose a reason for hiding this comment

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

Do you want me to remove all changes made to locale_fr-FR.ini?

Copy link
Member

Choose a reason for hiding this comment

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

yes

Copy link
Member

Choose a reason for hiding this comment

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

And don't hesitate to pass on crowdin to complete 🇫🇷 translation. ;-)

@sapk
Copy link
Member

sapk commented May 18, 2017

LGTM

@tboerger tboerger added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label May 18, 2017
@pgaskin
Copy link
Contributor

pgaskin commented May 18, 2017

LGTM

@tboerger tboerger added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels May 18, 2017
@lunny lunny modified the milestones: 1.2.0, 1.3.0 May 19, 2017
@lunny lunny merged commit 25d6e2a into go-gitea:master May 19, 2017
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. topic/ui Change the appearance of the Gitea UI type/enhancement An improvement of existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants