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

Update docs #720

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Update docs #720

wants to merge 3 commits into from
+97 −72

Conversation

y3n4
Copy link
Contributor

@y3n4 y3n4 commented Feb 17, 2025

  • smooth out development section
  • remove checkpassword
  • document how to use Dovecot Lua adapter

doobry-systemli
doobry-systemli reviewed Feb 23, 2025
View reviewed changes
Copy link
Contributor

@doobry-systemli doobry-systemli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really nice, thanks so much for this ❤️

Without testing it yet, I read through it and have some minor comments.

docs/installation/dovecot.md Outdated
}
```

Export the env vars to be picked up by Dovecots `import_environment` config:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would not understand right away how and where I have to export the env vars.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried to make the role of Dovecots import statement more clear

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

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

@doobry-systemli doobry-systemli doobry-systemli left review comments

@0x46616c6b 0x46616c6b Awaiting requested review from 0x46616c6b 0x46616c6b is a code owner

@t2d t2d Awaiting requested review from t2d t2d is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@y3n4 @doobry-systemli