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

Closed
wants to merge 6 commits into from
Closed

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

wants to merge 6 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 force-pushed the ci-auto-update-2.6.9 branch from 443e236 to 52e1ba7 Compare June 11, 2024 09:17
@Salamandar Salamandar mentioned this pull request Jun 11, 2024
Closed
2 tasks
@Salamandar
Copy link
Member

For reference: #195 (comment)

@Salamandar Salamandar force-pushed the ci-auto-update-2.6.9 branch from 46f5de2 to 95818a4 Compare June 11, 2024 09:29
@Salamandar Salamandar force-pushed the ci-auto-update-2.6.9 branch from 319e8d2 to 98b4786 Compare June 11, 2024 09:33
@Salamandar Salamandar mentioned this pull request Jun 11, 2024
Closed
@Salamandar Salamandar force-pushed the ci-auto-update-2.6.9 branch from 4c0f7c6 to 53cb6b6 Compare June 11, 2024 09:59
@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.

@Salamandar Salamandar changed the title Upgrade to v2.6.9 Upgrade to v2.6.9 - TODO: Fix LDAP Jun 20, 2024
@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

@Salamandar Salamandar force-pushed the ci-auto-update-2.6.9 branch from 98377f6 to d42c7bc Compare January 5, 2025 22:03
@Salamandar Salamandar force-pushed the ci-auto-update-2.6.9 branch from c119a21 to 61e5dfc Compare January 5, 2025 22:08
@Salamandar
Copy link
Member

Replacing it with #225 .

@Salamandar Salamandar closed this Jan 5, 2025
@Salamandar Salamandar deleted the ci-auto-update-2.6.9 branch January 5, 2025 22:24
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: Done
Milestone
No milestone
Development

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