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

Closing issues keywords should be configurable #7897

Closed
1 of 7 tasks
guillep2k opened this issue Aug 17, 2019 · 7 comments · Fixed by #8120
Closed
1 of 7 tasks

Closing issues keywords should be configurable #7897

guillep2k opened this issue Aug 17, 2019 · 7 comments · Fixed by #8120
Labels
type/enhancement An improvement of existing functionality

Comments

@guillep2k
Copy link
Member

  • Gitea version (or commit ref): v1.9
  • Git version: -
  • Operating system: -
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant
  • Log gist:

Description

Issue close keywords should be translated. Perhaps parsed according to the PR creator's language.

See models/action.go issueCloseKeywords

@techknowlogick
Copy link
Member

techknowlogick commented Aug 17, 2019

We have WIP keywords be configurable via app.ini, perhaps something similar for close keywords?

(Edit: please note this is just a suggestion, and not saying this is 100% the way it should be done)

Another way would be to add close keywords to translation files and reference those?

@techknowlogick techknowlogick added type/enhancement An improvement of existing functionality modifies/translation labels Aug 17, 2019
@guillep2k
Copy link
Member Author

Using the translation files sounds more like the Gitea way.

The goal of this project is to provide the easiest, fastest, and most painless way of setting up a self-hosted Git service.

😊

@lunny
Copy link
Member

lunny commented Aug 17, 2019

@guillep2k This should not be on translation file but follow WIP keywords. For Chinese we prefer close the issue via all these keywords close,closed,fix,fixed,resolve,resolved,关闭,修复,解决. A configuration item is better I think.

@guillep2k
Copy link
Member Author

I have no problem of going with a setting for this particular case.

@guillep2k
Copy link
Member Author

Thinking about it a little further, using translations will introduce a breaking change every time the translations are updated, so a setting is much better in that regard.

@lunny lunny changed the title Closing issues keywords should be translatable Closing issues keywords should be configurable Aug 18, 2019
@sapk
Copy link
Member

sapk commented Sep 16, 2019

For an other PR, what about adding a advanced part at the installation page allowing to auto setup specific local config and filled them from translation file?

@guillep2k
Copy link
Member Author

@sapk Sounds nice. We should make a TODO list of features to implement on that page.

@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type/enhancement An improvement of existing functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants