-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
website/integrations: add outline knowledge base #8786
website/integrations: add outline knowledge base #8786
Conversation
✅ Deploy Preview for authentik-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for authentik-storybook canceled.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8786 +/- ##
===========================================
+ Coverage 46.62% 92.40% +45.77%
===========================================
Files 626 634 +8
Lines 30996 31310 +314
===========================================
+ Hits 14451 28931 +14480
+ Misses 16545 2379 -14166
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Thanks so much @fallenhitokiri for this contribution! I made a few suggestions/edits, and also you will need to edit the |
Hi again @fallenhitokiri will you have a chance to make the requested changes, and then we can merge it? Or, if you're ok with it, I can make the changes and merge it. |
@tanberry I got started on the changes and expect to land them tomorrow or Tuesday. (I didn’t get a notification and only saw your comments by accident when checking my GitHub inbox) I used the HedgeDoc documentation as a template. Would some of the changes suggested here also apply to the HedgeDoc doc? |
Hi @fallenhitokiri thanks for the update, and I will take a look at the HedgeDoc docs as well. We do have a proper template there in the repo, but we need to do a better job of making it easier to find, for sure. We will look forward to your changes; let me know if we can help! |
Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com> Signed-off-by: Timo Zimmermann <timo@screamingatmyscreen.com>
Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com> Signed-off-by: Timo Zimmermann <timo@screamingatmyscreen.com>
Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com> Signed-off-by: Timo Zimmermann <timo@screamingatmyscreen.com>
Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com> Signed-off-by: Timo Zimmermann <timo@screamingatmyscreen.com>
Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com> Signed-off-by: Timo Zimmermann <timo@screamingatmyscreen.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @fallenhitokiri !!
* main: root: fix missing imports after #9081 (#9106) root: move database calls from ready() to dedicated startup signal (#9081) web: fix console log leftover (#9096) web: bump the eslint group in /web with 2 updates (#9098) core: bump twilio from 9.0.2 to 9.0.3 (#9103) web: bump the eslint group in /tests/wdio with 2 updates (#9099) core: bump drf-spectacular from 0.27.1 to 0.27.2 (#9100) core: bump django-model-utils from 4.4.0 to 4.5.0 (#9101) core: bump ruff from 0.3.4 to 0.3.5 (#9102) website/docs: update notes on SECRET_KEY (#9091) web: fix broken locale compile (#9095) website/integrations: add outline knowledge base (#8786) website/docs: fix typo (#9082) website/docs: email stage: fix example translation error (#9048)
Details
Adds instructions to setup OIDC for outline.
Checklist
If applicable
make website
)