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

Enable ldap #38

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

Enable ldap #38

wants to merge 5 commits into from

Conversation

Josue-T
Copy link
Contributor

@Josue-T Josue-T commented Apr 30, 2024

Problem

  • LDAP not configured and if enabled don't work correctly

Solution

  • Rework fully the LDAP config and use the new feature of permissions to map directly the permissions with access into Coin.

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)

@Josue-T Josue-T changed the base branch from master to testing April 30, 2024 20:22
@Josue-T Josue-T requested a review from alexAubin April 30, 2024 20:22
@Josue-T
Copy link
Contributor Author

Josue-T commented Apr 30, 2024

!testme

@yunohost-bot
Copy link
Contributor

🚀
Test Badge

@yunohost-bot
Copy link
Contributor

🐛
Test Badge

@Josue-T
Copy link
Contributor Author

Josue-T commented May 1, 2024

!testme

@yunohost-bot
Copy link
Contributor

🎠
Test Badge

@yunohost-bot
Copy link
Contributor

📚 🪱
Test Badge

@Salamandar Salamandar force-pushed the testing branch 2 times, most recently from ed61f2f to 113d30f Compare May 28, 2024 21:51
@Josue-T
Copy link
Contributor Author

Josue-T commented Jun 29, 2024

!testme

@yunohost-bot
Copy link
Contributor

📚 🐛
Test Badge

@yunohost-bot
Copy link
Contributor

🚀
Test Badge

@Josue-T Josue-T requested review from zamentur and alexAubin and removed request for alexAubin and ericgaspar June 29, 2024 21:43
@Josue-T
Copy link
Contributor Author

Josue-T commented Jul 8, 2024

!testme

@yunohost-bot
Copy link
Contributor

🌻
Test Badge

@yunohost-bot
Copy link
Contributor

📖
Test Badge

@Josue-T Josue-T requested review from alexAubin and removed request for zamentur and alexAubin July 8, 2024 10:11
@Josue-T
Copy link
Contributor Author

Josue-T commented Jul 8, 2024

@alexAubin @zamentur can you review, thanks 🙏

Base automatically changed from testing to master August 5, 2024 16:43
@Josue-T Josue-T changed the base branch from master to testing January 9, 2025 20:23
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

Successfully merging this pull request may close these issues.

2 participants