Skip to content

Conversation

@fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Feb 25, 2025

One single dialog was used to change the IoT Box hostname and connect it to a database.
These dialogs have been splitted.
Additionnaly, the hostname update is now performed through python instead of bash.

image
image

Forward-Port-Of: #198785

@robodoo
Copy link
Contributor

robodoo commented Feb 25, 2025

Pull request status dashboard

@fw-bot
Copy link
Contributor Author

fw-bot commented Feb 25, 2025

@loouis-t @Ysoroko cherrypicking of pull request #198785 failed.

stdout:

Auto-merging addons/hw_posbox_homepage/controllers/homepage.py
CONFLICT (content): Merge conflict in addons/hw_posbox_homepage/controllers/homepage.py

Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?).

In the former case, you may want to edit this PR message as well.

⚠️ after resolving this conflict, you will need to merge it via @robodoo.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@robodoo robodoo added forwardport This PR was created by @fw-bot conflict There was an error while creating this forward-port PR labels Feb 25, 2025
@C3POdoo C3POdoo added the RD research & development, internal work label Feb 25, 2025
One single dialog was used to change the IoT Box hostname and connect it to a database.
These dialogs have been splitted.
Additionnaly, the hostname update is now performed through python instead of bash.
Each dialog had different button colors, positions, input placholders or labels.
They now follow the same logic: placeholders and buttons in footer.
@loouis-t loouis-t force-pushed the saas-18.2-saas-18.1-hw_posbox_homepage-split-rename-and-server-lotr-417905-fw branch from 40665c5 to ba7178b Compare February 26, 2025 08:04
@loouis-t
Copy link
Contributor

@robodoo r+

@C3POdoo C3POdoo requested a review from a team February 26, 2025 08:06
@robodoo
Copy link
Contributor

robodoo commented Feb 26, 2025

@loouis-t @Ysoroko 'ci/security' failed on this reviewed PR.

@loouis-t
Copy link
Contributor

@odoo/rd-security hello security team, could you override this forward port please? (Here is the last override reason)

@xmo-odoo
Copy link
Collaborator

@robodoo override=ci/security

robodoo pushed a commit that referenced this pull request Feb 26, 2025
One single dialog was used to change the IoT Box hostname and connect it to a database.
These dialogs have been splitted.
Additionnaly, the hostname update is now performed through python instead of bash.

Part-of: #199327
Signed-off-by: Yaroslav Soroko (yaso) <yaso@odoo.com>
Signed-off-by: Louis Travaux (lotr) <lotr@odoo.com>
robodoo pushed a commit that referenced this pull request Feb 26, 2025
Each dialog had different button colors, positions, input placholders or labels.
They now follow the same logic: placeholders and buttons in footer.

closes #199327

Signed-off-by: Yaroslav Soroko (yaso) <yaso@odoo.com>
Signed-off-by: Louis Travaux (lotr) <lotr@odoo.com>
@robodoo robodoo closed this Feb 26, 2025
@xmo-odoo xmo-odoo deleted the saas-18.2-saas-18.1-hw_posbox_homepage-split-rename-and-server-lotr-417905-fw branch March 7, 2025 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conflict There was an error while creating this forward-port PR forwardport This PR was created by @fw-bot RD research & development, internal work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants