OCA / openupgradelib

A library with support functions to be called from Odoo migration scripts.
GNU Affero General Public License v3.0
85 stars 173 forks source link

[FIX+ADD] openupgradelib documentation #318

Closed legalsylvain closed 1 year ago

legalsylvain commented 1 year ago

Ref : https://github.com/OCA/OpenUpgrade/issues/3862

It replace the previous documentation that has not been updated since 2015. (here : https://openupgradelib.readthedocs.io/en/latest/)

Related to : https://github.com/OCA/OpenUpgrade/pull/3863

Once accepted, before merge :

After merge : set correctly github page in the setting configuration. (I don't have access but I guess @pedrobaeza / @StefanRijnhart have access)

legalsylvain commented 1 year ago

Hi @OCA/openupgrade-maintainers. Someone has an idea about that error ?

Log : https://app.travis-ci.com/github/OCA/openupgradelib/builds/262883178#L482 Travis line : https://github.com/OCA/openupgradelib/blob/master/.travis.yml#L35

At a first sight, it's look unrelated to the current PR.

hbrunn commented 1 year ago

seems indeed unrelated, looks more like one of those libs isn't compatible with py3 and needs to be version pinned?

legalsylvain commented 1 year ago

seems indeed unrelated, looks more like one of those libs isn't compatible with py3 and needs to be version pinned?

yes indeed. I'm not very confortable with the technical .travis.yml file and I have not a lot of time for the time being to invesetigate. I opened this issue : https://github.com/OCA/openupgradelib/issues/321 Any help welcome.

Thanks !

legalsylvain commented 1 year ago

/ocabot rebase

OCA-git-bot commented 1 year ago

Congratulations, PR rebased to master.

pedrobaeza commented 1 year ago

I suppose this requires a manual rebase removing the generated docs + the temp commit.

legalsylvain commented 1 year ago

Hi. Indeed !

@pedrobaeza : could you merge and set up the repo ? thanks !

pedrobaeza commented 1 year ago

Which setup I have to do?

legalsylvain commented 1 year ago

Hi @pedrobaeza.

enable github pages. (I guess disabled for the time being, but I don't have access to this part).

Go to the repository > settings > Pages

Then :

image

Once validated, a github action should be executed, and deploy the content of this new folder (https://github.com/OCA/openupgradelib/tree/master/docs) in https://oca.github.io/openupgradelib

let me know once it is done.

Thanks !

pedrobaeza commented 1 year ago

Ouch, I don't have access to Settings either.

@sbidoul @simahawk can you give us access to that part? @hbrunn @StefanRijnhart maybe you can access such part?

hbrunn commented 1 year ago

neither here nor for openupgrade, seems they both should be owned by @OCA/openupgrade-maintainers ?

pedrobaeza commented 1 year ago

I think so... I hope @OCA/board can fix this.

sbidoul commented 1 year ago

Due to the way the GitHub teams are synced from the OCA database, only the OCA representative has access to the admin settings. In this case it is @StefanRijnhart.

sbidoul commented 1 year ago

In the meantime I have configure gh pages as requested in https://github.com/OCA/openupgradelib/pull/318#issuecomment-1548445041

StefanRijnhart commented 1 year ago

Does anything still need to be done by me? The docs seem to be up, but I don't see any actual method documentation, only the headers: https://oca.github.io/openupgradelib/API.html#general-methods (looks like that might be fixed in https://github.com/OCA/openupgradelib/pull/324)

Relatedly, because of my large periods of absence from community projects maybe someone else wants to become the OCA representative?