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 v2.6.9 - TODO: Fix LDAP #203

Open
wants to merge 35 commits into
base: testing
Choose a base branch
from
Open

Upgrade to v2.6.9 - TODO: Fix LDAP #203

wants to merge 35 commits into from

Conversation

yunohost-bot
Copy link
Contributor

@yunohost-bot yunohost-bot commented May 13, 2024 •
edited by Salamandar
Loading

Upgrade to v2.6.9
https://github.com/wallabag/wallabag/releases/tag/2.6.9

  • Fix LDAP
  • Check out mbstring and polyfill missing dependencies?

@yunohost-bot
Copy link
Contributor Author

Fingers crossed!
Test Badge

@Salamandar Salamandar mentioned this pull request Jun 11, 2024
Closed
2 tasks
@Salamandar
Copy link
Member

For reference: #195 (comment)

@Salamandar Salamandar mentioned this pull request Jun 11, 2024
Closed
@Salamandar
Copy link
Member

LDAP will be fixed by ./composer.phar require symfony/ldap

@mcint
Copy link

mcint commented Jun 16, 2024 •
edited
Loading

🎉 Can we test this, merge into testing (for beta use), and possibly promote to master?

I'm unclear what else needs to be handled, after following closed #166 and #168, succeeded by #171 merge into testing.

It looks like my #162 and related function not available after php version upgrade might I suspect be fixed by a composer update or explicit inclusion of the smyfony/polyfill-mbstring package.

@Salamandar
Copy link
Member

No it's absolutely not testable as is, LDAP is still broken :'(

@Salamandar
Copy link
Member

But I take note on the two missing composer dependencies, i'll check it out.

@ericgaspar
Copy link
Member

!testme

@yunohost-bot
Copy link
Contributor Author

🎠
Test Badge

@yunohost-bot
Copy link
Contributor Author

📚
Test Badge

@lapineige
Copy link
Member

!testme

@yunohost-bot
Copy link
Contributor Author

📚 🪱
Test Badge

@yunohost-bot
Copy link
Contributor Author

Fingers crossed!
Test Badge

@ericgaspar
Copy link
Member

!testme

@yunohost-bot
Copy link
Contributor Author

😜
Test Badge

@yunohost-bot
Copy link
Contributor Author

🐛
Test Badge

@lapineige
Copy link
Member

55659 WARNING ./install: line 42: ynh_install_composer: command not found

But helpers are loaded ? 🤔

@lapineige
Copy link
Member

!testme

@yunohost-bot
Copy link
Contributor Author

🌻
Test Badge

@yunohost-bot
Copy link
Contributor Author

📖 🪱
Test Badge

@lapineige
Copy link
Member

(relaunched CI)

@lapineige
Copy link
Member

That ynh_composer issue 🤔

@JimboJoe
Copy link
Contributor

JimboJoe commented Nov 1, 2024

!testme

1 similar comment
@lapineige
Copy link
Member

!testme

@yunohost-bot
Copy link
Contributor Author

😜
Test Badge

@yunohost-bot
Copy link
Contributor Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
work needed
Projects
General
Status: To do
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@yunohost-bot @Salamandar @mcint @ericgaspar @lapineige @JimboJoe