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

core: fix rac property mapping requiring enterprise #8267

Merged

Conversation

BeryJu
Copy link
Member

@BeryJu BeryJu commented Jan 23, 2024

Details

REPLACE ME


Checklist

  • Local tests pass (ak test authentik/)
  • The code has been formatted (make lint-fix)

If an API change has been made

  • The API schema has been updated (make gen-build)

If changes to the frontend have been made

  • The code has been formatted (make web)
  • The translation files have been updated (make i18n-extract)

If applicable

  • The documentation has been updated
  • The documentation has been formatted (make website)

@BeryJu BeryJu requested a review from a team as a code owner January 23, 2024 11:01
Copy link

netlify bot commented Jan 23, 2024

Deploy Preview for authentik-storybook canceled.

Name Link
🔨 Latest commit c52cb10
🔍 Latest deploy log https://app.netlify.com/sites/authentik-storybook/deploys/65afa4c2d5f52c0008776bd0

Copy link

codecov bot commented Jan 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (53cbc64) 92.29% compared to head (8da34ec) 92.32%.

❗ Current head 8da34ec differs from pull request most recent head c52cb10. Consider uploading reports for the commit c52cb10 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8267      +/-   ##
==========================================
+ Coverage   92.29%   92.32%   +0.02%     
==========================================
  Files         608      608              
  Lines       30265    30265              
==========================================
+ Hits        27934    27941       +7     
+ Misses       2331     2324       -7     
Flag Coverage Δ
e2e 50.61% <ø> (+0.03%) ⬆️
integration 26.39% <ø> (ø)
unit 89.50% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Jens Langhammer <jens@goauthentik.io>
@BeryJu BeryJu force-pushed the enterprise/providers/rac/fix-mapping-requiring-enterprise branch from 8da34ec to c52cb10 Compare January 23, 2024 11:36
@BeryJu BeryJu merged commit f8d992d into main Jan 23, 2024
45 of 46 checks passed
@BeryJu BeryJu deleted the enterprise/providers/rac/fix-mapping-requiring-enterprise branch January 23, 2024 11:36
kensternberg-authentik added a commit that referenced this pull request Jan 23, 2024
* main:
  translate: Updates for file web/xliff/en.xlf in fr (#8272)
  translate: Updates for file locale/en/LC_MESSAGES/django.po in fr (#8271)
  fix codeowners
  core: bump goauthentik.io/api/v3 from 3.2023106.3 to 3.2023106.4
  web: bump API Client version (#8269)
  root: Multi-tenancy (#7590)
  website/docs: add helm chart 2024.1 breaking changes
  core: fix rac property mapping requiring enterprise (#8267)
  web: bump the wdio group in /tests/wdio with 4 updates (#8262)
  web: bump the eslint group in /tests/wdio with 2 updates (#8261)
  web: bump the eslint group in /web with 2 updates (#8263)
  core: bump uvicorn from 0.26.0 to 0.27.0 (#8264)
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.

1 participant