feat: dockge set/update agent friendly name #414

lohrbini · 2024-02-01T07:29:20Z

⚠️⚠️⚠️ 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]:

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)
My code follows the style guidelines of this project
I ran ESLint and other linters for modified files
I have performed a self-review of my own code and tested it
I have commented my code, particularly in hard-to-understand areas
(including JSDoc for methods)
My changes generate no new warnings
My code needed automated testing. I have added them (this is optional task)

Screenshots

Add the option to set a friendly name to a dockge remote agent. If the field is not used the `agent.url` will be displayed

cyril59310 · 2024-02-01T18:10:57Z

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

lohrbini · 2024-02-01T18:33:31Z

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 ..

cyril59310 · 2024-02-01T18:49:02Z

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.

adding `Friendly Name` to en.json

allow updates and removal of `friendly name`

lohrbini · 2024-02-01T21:35:38Z

@cyril59310 point 2 of your request is now also done. Only need to update codebase and remove all debug options

frontend/src/mixins/socket.ts

adding the `friendlyname` option to the running and paused container

cyril59310 · 2024-02-02T17:12:42Z

Thank you for taking my feedback.
I noticed afterwards that here too, the friendly name could be added.

when create a new stack via UI the friendly name should also be displayed

lohrbini · 2024-02-03T15:04:13Z

Thanks again @cyril59310 . This is now also implemented

Merge branch 'feat/dockge-agents-friendly-name' into feat/dockge-agents-friendly-name-editable

add a query to update the friendlyname correctly

remove edit-pen for controller which cannot have a friendlyname for now

lohrbini · 2024-02-06T07:03:34Z

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

fix linting issues

fix tsc check by updating var type

remove if condition and update to proper values to prevent duplicates

cyril59310 · 2024-02-06T17:53:01Z

I made some suggestions in order to add translation keys for our translators.

louislam · 2024-02-06T21:59:01Z

backend/migrations/2023-12-20-2117-agent-table.ts

@@ -7,6 +7,7 @@ export async function up(knex: Knex): Promise<void> {
        table.string("url", 255).notNullable().unique();
        table.string("username", 255).notNullable();
        table.string("password", 255).notNullable();
+        table.string("friendlyname", 255);


This file cannot be modified. Have to create a new file.

Migration guide:
https://github.com/louislam/uptime-kuma/tree/master/db/knex_migrations

Also the naming convention, it should be friendly_name.

Will come back later, as I would like to finish the next milestone of Uptime Kuma first.

louislam · 2024-02-06T22:03:46Z

backend/agent-manager.ts

     */
-    async add(url : string, username : string, password : string) : Promise<Agent> {
+    async add(url : string, username : string, password : string, friendlyname : string) : Promise<Agent> {


Name Conventions

Javascript/Typescript: camelCaseType

SQLite: snake_case (Underscore)

CSS/SCSS: kebab-case (Dash)

https://github.com/louislam/dockge/blob/master/CONTRIBUTING.md#name-conventions

nathan815

Nice feature!

Have some code feedback

nathan815 · 2024-04-23T04:18:56Z

backend/agent-manager.ts

+     */
+
+    async update(friendlyname : string, updatedFriendlyName : string) {
+        await R.exec("UPDATE agent SET friendlyname = " + updatedFriendlyName + " WHERE friendlyname = " + friendlyname + "");


SQL injection vuln here - if name has quotes this will break.

You can use parameter bindings to build queries safely:

await R.exec("UPDATE agent SET friendlyname = ? WHERE friendlyname = ?", [ updatedFriendlyName, friendlyname ]);

https://github.com/louislam/redbean-node/blob/master/docs/Query.md

Instead of a raw query, you could utilize the redbean ORM methods. See: https://github.com/louislam/redbean-node/blob/master/docs/Create-Update-Bean.md

Something like this if by ID (primary key):

const agent = await R.load('agent', id); agent.friendlyName = friendlyName; await R.store(agent);

Or by URL:

const agent = await R.findOne('agent', ' url = ? ', [url]); agent.friendlyName = friendlyName; await R.store(agent);

nathan815 · 2024-04-23T04:31:25Z

backend/agent-manager.ts

+     * @param updatedFriendlyName
+     */
+
+    async update(friendlyname : string, updatedFriendlyName : string) {


I would consider accepting the agent's id or the url here for identifying the agent to update instead of friendly name.

ID is the primary key, so thats probably best to use for updating the agent. But URL is always set and has a unique constraint, so it will work too.

nathan815 · 2024-04-23T04:34:49Z

backend/models/agent.ts

+            friendlyname: this.friendlyname,
+            updatedFriendlyName: this.updatedFriendlyName


Why is updatedFriendlyName also here on the model? Shouldn't just friendlyName be sufficient?

lohrbini · 2024-05-22T19:12:16Z

Long time no see. Will have a look the next days on all your feedback. Many thanks 🙏🏻

feat: dockge agent friendly name

c0fc76a

Add the option to set a friendly name to a dockge remote agent. If the field is not used the `agent.url` will be displayed

lohrbini added 2 commits February 1, 2024 18:51

docs(en): add missing new item

11f5fff

adding `Friendly Name` to en.json

DRAFT: feat(friendlyName): make friendlyname configurable

54e158f

allow updates and removal of `friendly name`

larswmh reviewed Feb 2, 2024

View reviewed changes

frontend/src/mixins/socket.ts Outdated Show resolved Hide resolved

lohrbini force-pushed the feat/dockge-agents-friendly-name branch from 6529f6d to 96b5205 Compare February 2, 2024 06:17

feat(containerTerminal): add friendly name

8f142af

adding the `friendlyname` option to the running and paused container

lohrbini force-pushed the feat/dockge-agents-friendly-name branch from 96b5205 to 8f142af Compare February 2, 2024 06:20

feat(compose): add friendly names

62ceb44

when create a new stack via UI the friendly name should also be displayed

lohrbini added 3 commits February 6, 2024 06:53

chore(deps): update from upstream

5885665

Merge branch 'feat/dockge-agents-friendly-name' into feat/dockge-agents-friendly-name-editable

fix(agent-manager): add update query

2b64b3f

add a query to update the friendlyname correctly

fix(DashboardHome): remove edit-pen for controller

a8b75fb

remove edit-pen for controller which cannot have a friendlyname for now

lohrbini changed the title ~~feat: dockge agent friendly name~~ feat: dockge set/update agent friendly name Feb 6, 2024

lohrbini added 3 commits February 6, 2024 07:30

style(agent-manager): liniting

9748469

fix linting issues

fix(manage-agent-socket-handler): type

052cf17

fix tsc check by updating var type

fix(DashboardHome): duplicate code

dcc48d3

remove if condition and update to proper values to prevent duplicates

lohrbini force-pushed the feat/dockge-agents-friendly-name branch from 1e688f8 to dcc48d3 Compare February 6, 2024 07:55

louislam reviewed Feb 6, 2024

View reviewed changes

louislam added this to the 1.5.0 milestone Feb 6, 2024

louislam added the question Further information is requested label Feb 6, 2024

louislam reviewed Feb 6, 2024

View reviewed changes

nathan815 reviewed Apr 23, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: dockge set/update agent friendly name #414

feat: dockge set/update agent friendly name #414

lohrbini commented Feb 1, 2024 •

edited

cyril59310 commented Feb 1, 2024 •

edited

lohrbini commented Feb 1, 2024 •

edited

cyril59310 commented Feb 1, 2024

lohrbini commented Feb 1, 2024 •

edited

cyril59310 commented Feb 2, 2024

lohrbini commented Feb 3, 2024

lohrbini commented Feb 6, 2024 •

edited

cyril59310 commented Feb 6, 2024 •

edited

louislam Feb 6, 2024 •

edited

louislam Feb 6, 2024

nathan815 left a comment

nathan815 Apr 23, 2024

nathan815 Apr 23, 2024

nathan815 Apr 23, 2024

nathan815 Apr 23, 2024

lohrbini commented May 22, 2024

		friendlyname: this.friendlyname,
		updatedFriendlyName: this.updatedFriendlyName

feat: dockge set/update agent friendly name #414

Are you sure you want to change the base?

feat: dockge set/update agent friendly name #414

Conversation

lohrbini commented Feb 1, 2024 • edited

Disclaimer

Description

Type of change

Screenshots

cyril59310 commented Feb 1, 2024 • edited

lohrbini commented Feb 1, 2024 • edited

cyril59310 commented Feb 1, 2024

lohrbini commented Feb 1, 2024 • edited

cyril59310 commented Feb 2, 2024

lohrbini commented Feb 3, 2024

lohrbini commented Feb 6, 2024 • edited

cyril59310 commented Feb 6, 2024 • edited

louislam Feb 6, 2024 • edited

Choose a reason for hiding this comment

louislam Feb 6, 2024

Choose a reason for hiding this comment

nathan815 left a comment

Choose a reason for hiding this comment

nathan815 Apr 23, 2024

Choose a reason for hiding this comment

nathan815 Apr 23, 2024

Choose a reason for hiding this comment

nathan815 Apr 23, 2024

Choose a reason for hiding this comment

nathan815 Apr 23, 2024

Choose a reason for hiding this comment

lohrbini commented May 22, 2024

lohrbini commented Feb 1, 2024 •

edited

cyril59310 commented Feb 1, 2024 •

edited

lohrbini commented Feb 1, 2024 •

edited

lohrbini commented Feb 1, 2024 •

edited

lohrbini commented Feb 6, 2024 •

edited

cyril59310 commented Feb 6, 2024 •

edited

louislam Feb 6, 2024 •

edited