faq: mention EMAIL_DOMAIN_WHITELIST on allowing email domains

[Mikaela]

I wanted to only allow two domains related to two specific organisations and followed the FAQ and was confused until reading app.ini more closely and finding out that the FAQ was wrong.

Questions that aren't yet clear to me:

• Why does this section talk about email domains, while it documented only OpenID? Should I move it to a separate section?
• I didn't see any blacklist in app.ini, should I remove /block?

[codecov-io]

Codecov Report

Merging #8198 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #8198      +/-   ##
==========================================
+ Coverage    41.6%   41.61%   +<.01%     
==========================================
  Files         483      483              
  Lines       64741    64741              
==========================================
+ Hits        26937    26940       +3     
+ Misses      34355    34354       -1     
+ Partials     3449     3447       -2

Impacted Files	Coverage Δ
models/unit.go	62.16% <0%> (-5.41%)	⬇️
models/repo_list.go	74.14% <0%> (+0.97%)	⬆️
modules/process/manager.go	81.15% <0%> (+4.34%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a372363...829f3a3. Read the comment docs.

[jolheiser]

Ah, that may have been an oversight. Thanks for the update!

1. If you could move it into another sub-section, reference the [openid] section of the app.ini, and update the TOC, that would be great.
2. Yes, for email whitelist you can remove the /block portion

[Mikaela]

Thanks for the fast response. Does this look OK (assuming the ToC update fixed the build)?

Also should I merge the commits or do you usually squash and merge?

[Mikaela]

I don't understand what drone wishes me to do?

[jolheiser]

It looks like the Drone issue is unrelated to this PR.

As for merging, we squash commits when we merge, so no worries.

[jolheiser]

Thanks again for the docs PR!