Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compatibility with NC 29 #397

Open
cedricbonhomme opened this issue Jul 3, 2024 · 4 comments
Open

Compatibility with NC 29 #397

cedricbonhomme opened this issue Jul 3, 2024 · 4 comments

Comments

@cedricbonhomme
Copy link

Describe the bug

Gestion is displayed as not compatible with Nextcloud 29. Is there any reason for this ?

@cedricbonhomme
Copy link
Author

if I force the update there is SQLSTATE error:

cedric@ssh:~/www/membres$ php occ app:install gestion
The process control (PCNTL) extensions are required in case you want to interrupt long running commands - see https://www.php.net/manual/en/book.pcntl.php
Additionally the function 'pcntl_signal' and 'pcntl_signal_dispatch' need to be enabled in your php.ini.
Error: Database error when running migration 20300Date20230714134445 for app gestion
An exception occurred while executing a query: SQLSTATE[23502]: Not null violation: 7 ERROR:  column "id_configuration" of relation "oc_gestion_client" contains null values

but I cannot even find the column id_configuration. So I guess it's due to a wrong migration.

@cedricbonhomme
Copy link
Author

ok, for now I hacked the migration script. I added 'notnull => false' here and here. Then the upgrade script worked. And there are correct values in the column id_configuration in the different tables.

@baimard
Copy link
Owner

baimard commented Jul 4, 2024

Merci Cédric, dès que j'ai un instant je regarde. Je n'avais pas encore validé le passage en NC 29. Les release de NC sont vraiment rapide !

@Thatoo
Copy link

Thatoo commented Aug 21, 2024

When compatibility will be ok, could you mention it here (when closing the issue) for us to know we can update to NC 29 please?
Thank you

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

No branches or pull requests

3 participants