Skip to content

Conversation

@carlos-lopez-tecnativa
Copy link
Contributor

@carlos-lopez-tecnativa carlos-lopez-tecnativa commented Mar 11, 2025

Supersedes #4590
Rebase done

TT49872
@Tecnativa @pedrobaeza @rvalyi @hbrunn @MiquelRForgeFlow could you please review this?

@legalsylvain
Copy link
Contributor

I don't get why supersed the other PR. Let @acpMicrocom finish his PR.
you requested a change only a few days ago.
Note : there is no urgence because this PR can not be merged. (web_hierarchy is not done).

@carlos-lopez-tecnativa
Copy link
Contributor Author

I don't get why supersed the other PR. Let @acpMicrocom finish his PR. you requested a change only a few days ago. Note : there is no urgence because this PR can not be merged. (web_hierarchy is not done).

According to @pedrobaeza, @acpMicrocom has been inactive for several months, so their PRs can be superseded. That is why I created a new PR. This is not the first time I have pinged @acpMicrocom without receiving a response, and the same has happened in several other PRs."

@carlos-lopez-tecnativa
Copy link
Contributor Author

@legalsylvain I don’t understand what happened with web_hierarchy. This is a new module in V17 and didn’t exist before. What steps are necessary? Could you please clarify this?

@legalsylvain
Copy link
Contributor

Hi @carlos-lopez-tecnativa.

hr_org_chart depends no web_hierarchy . It's a new module, but it could requires some migration script.
modules have to be merged in OpenUpgrade, in order of dependency.
See : #4810

@carlos-lopez-tecnativa carlos-lopez-tecnativa deleted the 17.0_ou_hr_org_chart branch March 20, 2025 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants