Skip to content

Flows Stage 1#8

Merged
BeryJu merged 80 commits intomasterfrom
flows-stage1
May 14, 2020
Merged

Flows Stage 1#8
BeryJu merged 80 commits intomasterfrom
flows-stage1

Conversation

@BeryJu
Copy link
Member

@BeryJu BeryJu commented May 7, 2020

This is the first stage of flows, replacing #2. It implements flows with the current, View-based Factors, instead of migration to an API-based Design. This should make migration easier.

  • Get rid of AuthenticationView, replace with a simpler FlowExecutor
  • Rename factors to something more general as they will be used for more general tasks (stage?)
  • Move factors to flows/factors

@BeryJu BeryJu marked this pull request as ready for review May 8, 2020 16:28
@BeryJu BeryJu self-assigned this May 8, 2020
@codecov
Copy link

codecov bot commented May 13, 2020

Codecov Report

Merging #8 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master       #8   +/-   ##
=======================================
  Coverage   74.00%   74.00%           
=======================================
  Files         279      279           
  Lines        6759     6759           
=======================================
  Hits         5002     5002           
  Misses       1757     1757           

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 814c797...814c797. Read the comment docs.

@BeryJu BeryJu merged commit 8cfd3f9 into master May 14, 2020
@BeryJu BeryJu deleted the flows-stage1 branch May 15, 2020 09:05
@BeryJu BeryJu mentioned this pull request May 16, 2020
Closed
@BeryJu BeryJu added this to the Release 0.9 milestone Jun 4, 2020
tacerus added a commit to tacerus/authentik that referenced this pull request Feb 12, 2026
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