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

Ory Kratos v0.8 #2166

Merged
merged 21 commits into from
Feb 14, 2022
Merged

Ory Kratos v0.8 #2166

merged 21 commits into from
Feb 14, 2022

Conversation

aeneasr
Copy link
Member

@aeneasr aeneasr commented Jan 20, 2022

No description provided.

@aeneasr aeneasr requested a review from zepatrik as a code owner January 20, 2022 15:48
@aeneasr aeneasr force-pushed the v0.8 branch 2 times, most recently from 9ce1590 to f362323 Compare January 21, 2022 14:16
@aeneasr aeneasr mentioned this pull request Jan 24, 2022
7 tasks
@codecov
Copy link

codecov bot commented Feb 8, 2022

Codecov Report

Merging #2166 (de61ed0) into master (10778fd) will increase coverage by 0.65%.
The diff coverage is 93.12%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2166      +/-   ##
==========================================
+ Coverage   75.27%   75.93%   +0.65%     
==========================================
  Files         294      297       +3     
  Lines       15651    15905     +254     
==========================================
+ Hits        11782    12077     +295     
+ Misses       3037     2972      -65     
- Partials      832      856      +24     
Impacted Files Coverage Δ
cmd/daemon/serve.go 82.75% <50.00%> (+0.20%) ⬆️
courier/template/load_template.go 78.43% <86.53%> (+11.76%) ⬆️
courier/template/testhelpers/testhelpers.go 95.06% <95.06%> (ø)
cmd/identities/validate.go 62.12% <100.00%> (ø)
courier/courier.go 72.91% <100.00%> (+3.47%) ⬆️
courier/template/recovery_invalid.go 100.00% <100.00%> (ø)
courier/template/recovery_valid.go 100.00% <100.00%> (ø)
courier/template/stub.go 100.00% <100.00%> (ø)
courier/template/verification_invalid.go 100.00% <100.00%> (ø)
courier/template/verification_valid.go 100.00% <100.00%> (ø)
... and 60 more

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 10778fd...de61ed0. Read the comment docs.

aeneasr and others added 19 commits February 11, 2022 14:42
…_id` (#1964)

BREAKING CHANGES: A major issue has been lingering in the configuration for a while. What happens to your identities when you update a schema? The answer was, it depends on the change. If the change is incompatible, some things might break!

To resolve this problem we changed the way you define schemas. Instead of having a global `default_schema_url` which developers used to update their schema, you now need to define the `default_schema_id` which must reference schema ID in your config. To update your existing configuration, check out the patch example below:

```patch
identity:
-  default_schema_url: file://stub/identity.schema.json
+  default_schema_id: default
+  schemas:
+  - id: default
+    url: file://stub/identity.schema.json
```

Ideally, you would version your schema and update the `default_schema_id` with every change to the new version:

```yaml
identity:
  default_schema_id: user_v1
  schemas:
    - id: user_v0
      url: file://path/to/user_v0.json
    - id: user_v1
      url: file://path/to/user_v1.json
```
It is now possible to override individual courier email templates using the configuration system!

Closes #2054
@Benehiko Benehiko mentioned this pull request Feb 14, 2022
7 tasks
@aeneasr aeneasr merged commit 83a0ce7 into master Feb 14, 2022
@aeneasr aeneasr deleted the v0.8 branch February 14, 2022 18:47
@aeneasr aeneasr restored the v0.8 branch February 14, 2022 18:48
@aeneasr aeneasr deleted the v0.8 branch February 14, 2022 18:48
@aeneasr aeneasr restored the v0.8 branch February 14, 2022 18:48
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.

3 participants