project icon indicating copy to clipboard operation
project copied to clipboard

[15.0][MIG] project_key

Open vincent-hatakeyama opened this issue 1 year ago • 5 comments

Standard migration of the project_key module.

See #1055 for comments and history

vincent-hatakeyama avatar Mar 29 '24 15:03 vincent-hatakeyama

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

github-actions[bot] avatar Jul 28 '24 12:07 github-actions[bot]

LGTM

coleste avatar Nov 08 '24 23:11 coleste

/ocabot migration project_key

rousseldenis avatar Dec 06 '24 09:12 rousseldenis

@vincent-hatakeyama Could you improve commit message:

Like [IMP] project_key: ...? Thanks

That looked like a fix rather than an improvement. I’ve rebased and edited the commit message

vincent-hatakeyama avatar Dec 06 '24 18:12 vincent-hatakeyama

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

github-actions[bot] avatar Jun 15 '25 12:06 github-actions[bot]

@vincent-hatakeyama Could you rebase, please?

marcelsavegnago avatar Aug 13 '25 16:08 marcelsavegnago

Functional 👍

why are there commits after the migration commit?

First message indicates:

See https://github.com/OCA/project/pull/1055 for comments and history

It was suggested to port some fixes/changes in the original PR.

vincent-hatakeyama avatar Sep 05 '25 07:09 vincent-hatakeyama

Thanks for the feedback, but there’s no plan for us to work on this PR.

There’s two possibilities:

  • I close the PR and someone can use it as a basis for the migration of the module.
  • I keep it and someone opens a PR on our fork to update this.

If I don’t have any answer for a while, I’ll just close the PR.

vincent-hatakeyama avatar Sep 29 '25 07:09 vincent-hatakeyama

I just created #1570 which is a fork of your work plus the fix discussed recently.

jans23 avatar Sep 29 '25 07:09 jans23