feat: dockge set/update agent friendly name
⚠️⚠️⚠️ Since we do not accept all types of pull requests and do not want to waste your time. Please be sure that you have read pull request rules: https://github.com/louislam/dockge/blob/master/CONTRIBUTING.md
Tick the checkbox if you understand [x]:
- [x] I have read and understand the pull request rules.
Disclaimer
I am not a Software programmer and only have basic understandings on any Programming Languages used in the project, so please be patient and maybe assist me with further development of this feature
Description
Since dockge agents is still a beta feature and in the future maybe more and more remote instances will be added I wanted to enable the option for setting a friendly name for a remote instance. If the name is not set via the prompt. The default agent.url will be displayed
Fixes #345
Type of change
-
New feature (non-breaking change which adds functionality)
-
[x] My code follows the style guidelines of this project
-
[x] I ran ESLint and other linters for modified files
-
[x] I have performed a self-review of my own code and tested it
-
[x] I have commented my code, particularly in hard-to-understand areas (including JSDoc for methods)
-
[x] My changes generate no new warnings
-
[ ] My code needed automated testing. I have added them (this is optional task)
Screenshots
After testing your PR, I have two small remarks:
- Could you add the translation key "Friendly Name" to the en.json file?
- It would be interesting for the friendly name to be applied here as well
Hey @cyril59310,
thanks for your feedback.
The translation can be added - for sure.
~~Could you maybe point out, where the friendlyName is missing exactly ?~~ Figured out where you want to add the friendlyName
Currently trying to refactor the code a bit to enable renaming - which seems more difficult as expected ..
In the file
frontend/src/lang/en.json ,
please add "Friendly Name": "Friendly Name" so that translators can translate it into other languages using our Weblate tool.
@cyril59310 point 2 of your request is now also done. Only need to update codebase and remove all debug options
Thank you for taking my feedback.
I noticed afterwards that here too, the friendly name could be added.
Thanks again @cyril59310 . This is now also implemented
Finally I was able to get the update feature working. Friendly names are now updated properly by a small dialog
@larswmh maybe need help this time with syntaxing/linting issues for sqlite statement. Or even a better solution than a "hardcoded" query
I made some suggestions in order to add translation keys for our translators.
Long time no see. Will have a look the next days on all your feedback. Many thanks 🙏🏻
would love for this to be added!
Ready & happy about a second review :)
&& could need help with that one failing CI pipeline
on which tag do i need to be to have this feature?
on which tag do i need to be to have this feature?
It is planned to be included in the next release v1.5.0 . To use this feature you need to be on my branch and build the image on your own