search

yinho999 / loco-oauth2

9 stars 6 forks source link

[WIP] Loco v0.6 + JWT support #6

Closed yinho999 closed 4 months ago

yinho999 commented 4 months ago
BWStearns commented 4 months ago

Thanks for updating! I will give this a go on my repo tomorrow and report how it goes.

yinho999 commented 4 months ago

@BWStearns Thank you so much for your interest in helping and offering to work on this update. I appreciate your enthusiasm and willingness to contribute to this project.

I wanted to give you a quick heads up about this particular PR. It's a bit of a special case - I am dealing with an emergency fix for those recent loco breaking changes you mentioned. On top of that, it's tied up with some new features and critical security stuff. Because of all that, I need to handle this one myself to make sure we get it out quickly and safely.

I know it's a bummer that I can't include external contributions on this one, but please don't let that discourage you! Your input is super valuable, and I'd love to have you involved in future updates. Once I get this emergency fix out the door, I'd be thrilled to check out any ideas or improvements you've got.

BWStearns commented 4 months ago

All good! I appreciate you getting the fix out so fast. Going forward if I run into anything missing/improvable that I can see how to contribute back I will. Cheers!