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.

Sign up for GitHub

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

Jump to bottom

Upgrade to v0.9.5 TODO: Fix LDAP #99

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Upgrade to v0.9.5 TODO: Fix LDAP #99

wants to merge 12 commits into from

Conversation

tituspijean
Copy link
Member

PR Status

  • Code finished and ready to be reviewed/tested
  • The fix/enhancement were manually tested (if applicable)

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

@tituspijean
Copy link
Member Author

Upon trying to upgrade an existing instance of Baïkal, the page displays the following error:

Incomplete installation
Baïkal dependencies have not been installed. If you are a regular user, this means that you probably downloaded the wrong zip file.
To install the dependencies manually, execute "composer install" in the Baïkal root folder.

@alexAubin
Copy link
Member

Hmmm could be related to --full_replace=1 or the weird change in --keep ?

@ericgaspar
Copy link
Member

!testme

@yunohost-bot
Copy link

📖
Test Badge

@yunohost-bot
Copy link

😜
Test Badge

alexAubin
alexAubin reviewed Jun 9, 2024
View reviewed changes
manifest.toml Outdated Show resolved Hide resolved
@ericgaspar
Copy link
Member

!testme

@yunohost-bot
Copy link

🌻
Test Badge

@yunohost-bot
Copy link

🪱
Test Badge

@Salamandar Salamandar force-pushed the testing branch 2 times, most recently from 8261a90 to 2d93bcd Compare June 20, 2024 11:44
@Salamandar Salamandar changed the title Upgrade to v0.9.5, replace full source except config Upgrade to v0.9.5 Jun 20, 2024
@Salamandar
Copy link
Member

I got this on login:

Creation of dynamic property Baikal\Core\LDAPUserBindAuth::$currentUser is deprecated

@Salamandar Salamandar changed the title Upgrade to v0.9.5 Upgrade to v0.9.5 TODO: Fix LDAP Jun 20, 2024
yunohost-bot and others added 3 commits September 8, 2024 17:12
@yunohost-bot @ericgaspar
[autopatch] Automatic patch attempt for helpers 2.1 (#104)
f5824f2 
* [autopatch] Automatic patch attempt for helpers 2.1

* cleaning

---------

Co-authored-by: Yunohost-Bot <>
Co-authored-by: Éric Gaspar <46165813+ericgaspar@users.noreply.github.com>
@yunohost-bot
Auto-update READMEs
6848803
@ericgaspar
cleaning
46fbff3
@ericgaspar
Copy link
Member

!testme

@yunohost-bot
Copy link

🪱
Test Badge

@yunohost-bot
Copy link

✌️
Test Badge

@ericgaspar
Copy link
Member

!testme

@yunohost-bot
Copy link

Test it, book it, worm it, technologic. 🤖 🎶
Test Badge

@ericgaspar
Copy link
Member

!testme

@yunohost-bot
Copy link

Living in the past, are we? 🐮👁️
Test Badge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Salamandar Salamandar Awaiting requested review from Salamandar

@alexAubin alexAubin Awaiting requested review from alexAubin

Assignees
No one assigned
Labels
None yet
Projects
General
Status: To do
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@tituspijean @alexAubin @ericgaspar @yunohost-bot @Salamandar