Skip to content

Conversation

DeamonMV
Copy link

@DeamonMV DeamonMV commented May 3, 2024

  • updated to kratos v1.1.0
  • added kratos-admin-ui
  • renamed index.html to index.gohtml for better reflection of what it's and make possible to use gotemplater with Jetberains IDE
  • updated schema, added additionally required username and tos fields
  • fixed issue with update password and profile - it should be two different forms.

Hello @atreya2011 can you please help with person.schema.yaml and generating a person.schema.json unfortunately I didn't get a point of it. I hope you are well and will find a time for this.

And I guess I missed that main branch not main, but hydra-consent - but maybe it's can be used any way. For me right now hydra is not relevant.

Thank you

@atreya2011
Copy link
Owner

@DeamonMV

Thanks for this. I will take a look at the PR. Please give me some time 🙇🏼‍♂️

I recently changed the default branch because the majority of the people were interested in the Kratos-Hydra integration…

I created the YAML file for defining the schema because, personally, I felt it was easier to read the YAML than a very big JSON file.

I hope that answers your question.

@DeamonMV DeamonMV force-pushed the update-to-v1.1.1-with-ehansments branch from 4043214 to 2e54249 Compare May 5, 2024 10:06
- updated to kratos v1.1.0
- added kratos-admin-ui
- rename index.html to index.gohtml for better reflection of what it's and make possible to use gotemplater with Jetberains IDE
- updated schema, added additionally required username and tos fields
- fixed issue with update password and profile - it should two different forms.
@DeamonMV DeamonMV force-pushed the update-to-v1.1.1-with-ehansments branch from 2e54249 to 27e40a9 Compare May 5, 2024 10:12
@DeamonMV
Copy link
Author

DeamonMV commented May 7, 2024

Pushed small fixes to json schema and wrapped function inside (s *server)

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