Skip to content

Conversation

rissson
Copy link
Member

@rissson rissson commented Oct 9, 2025

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)

If applicable

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

@rissson rissson self-assigned this Oct 9, 2025
@rissson rissson requested a review from a team as a code owner October 9, 2025 11:55
Copy link

netlify bot commented Oct 9, 2025

Deploy Preview for authentik-storybook canceled.

Name Link
🔨 Latest commit f98da92
🔍 Latest deploy log https://app.netlify.com/projects/authentik-storybook/deploys/68e7b54b6157490008946368

Copy link

netlify bot commented Oct 9, 2025

Deploy Preview for authentik-docs canceled.

Name Link
🔨 Latest commit f98da92
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/68e7b54b06d38a0008ba94da

Copy link

netlify bot commented Oct 9, 2025

Deploy Preview for authentik-integrations canceled.

Name Link
🔨 Latest commit f98da92
🔍 Latest deploy log https://app.netlify.com/projects/authentik-integrations/deploys/68e7b54b5ece310008a5528f

Copy link

codecov bot commented Oct 9, 2025

Codecov Report

❌ Patch coverage is 79.16667% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.92%. Comparing base (4777bbd) to head (f98da92).
⚠️ Report is 10 commits behind head on main.

Files with missing lines Patch % Lines
authentik/lib/sync/outgoing/api.py 40.00% 3 Missing ⚠️
authentik/outposts/signals.py 71.42% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #17356      +/-   ##
==========================================
- Coverage   92.97%   92.92%   -0.06%     
==========================================
  Files         857      857              
  Lines       46563    46597      +34     
==========================================
+ Hits        43290    43298       +8     
- Misses       3273     3299      +26     
Flag Coverage Δ
e2e 45.91% <41.66%> (-0.13%) ⬇️
integration 23.11% <33.33%> (-0.01%) ⬇️
unit 91.04% <79.16%> (+<0.01%) ⬆️
unit-migrate 91.10% <79.16%> (+<0.01%) ⬆️

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.

@rissson
Copy link
Member Author

rissson commented Oct 9, 2025

yeah so this is not working

@rissson rissson marked this pull request as draft October 9, 2025 12:19
Signed-off-by: Marc 'risson' Schmitt <[email protected]>
@rissson rissson force-pushed the tasks-eagerly-save-messages branch from c591c02 to c2632b5 Compare October 9, 2025 12:58
@rissson rissson changed the title tasks: eagerly save uid and messages tasks: set uid early Oct 9, 2025
@rissson rissson marked this pull request as ready for review October 9, 2025 12:59
@rissson rissson requested a review from BeryJu October 9, 2025 12:59
Signed-off-by: Marc 'risson' Schmitt <[email protected]>
@rissson rissson enabled auto-merge (squash) October 9, 2025 13:00
Signed-off-by: Marc 'risson' Schmitt <[email protected]>
@rissson rissson merged commit 6341252 into main Oct 9, 2025
94 of 97 checks passed
@rissson rissson deleted the tasks-eagerly-save-messages branch October 9, 2025 13:41
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