Skip to content

Conversation

aeneasr
Copy link
Member

@aeneasr aeneasr commented Aug 27, 2025

Copy link
Member

@vinckr vinckr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice!
a few style changes, otherwise looks gtg

@aeneasr
Copy link
Member Author

aeneasr commented Aug 27, 2025

Thank you! Maybe we can somehow add a claude.md or some other guidelines to the repo so that ai tooling can help make these rules more strict, since i used it to improve the writing! Would probably make everyones life easier :)

aeneasr and others added 3 commits August 27, 2025 15:13
Co-authored-by: Vincent <vincent@ory.sh>
Co-authored-by: Vincent <vincent@ory.sh>
Co-authored-by: Vincent <vincent@ory.sh>
@vinckr
Copy link
Member

vinckr commented Aug 27, 2025

Thank you! Maybe we can somehow add a claude.md or some other guidelines to the repo so that ai tooling can help make these rules more strict, since i used it to improve the writing! Would probably make everyones life easier :)

sure! happy to do it, do you know a "standard" way to do it?
I tried out giving the LLM the readme as context but it still ignored the bolding rules in my test... never tried with claude though.

aeneasr and others added 4 commits August 27, 2025 15:18
Co-authored-by: Vincent <vincent@ory.sh>
Co-authored-by: Vincent <vincent@ory.sh>
Co-authored-by: Vincent <vincent@ory.sh>
Co-authored-by: Vincent <vincent@ory.sh>
@aeneasr
Copy link
Member Author

aeneasr commented Aug 27, 2025

I tried out giving the LLM the readme as context but it still ignored the bolding rules in my test... never tried with claude though.

not really, also for chatgpt not easy. for claude we can do claude.md which is used for context but yeah it often sometimes ignore

Updated CORS configuration to allow all origins for public OAuth2 client registration.
Copy link
Member

@vinckr vinckr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@vinckr vinckr merged commit 4955882 into master Aug 28, 2025
8 checks passed
@vinckr vinckr deleted the fix-cors branch August 28, 2025 14:40
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.

Hydra 2 does not send CORS headers in response to OPTIONS preflight request

2 participants