kitsu-web

kitsu-web copied to clipboard

👢 🗑️

I've observed an issue here where moment timezone doesn't seem to get imported properly and it breaks the _getCurrentUser method in the application route. Not sure if it's something weird on my end so would appreciate if whoever reviews this could check if that works for them. Dunno if the discussion here is related: https://github.com/hummingbird-me/kitsu-web/pull/775

Thank you for doing this!

SSO (with nolt('identify', { jwt: token })) doesn't seem to happen when logged in on Kitsu

Edit: Nvm, looks like it's calling a query that hasn't been merged/deployed on production

Ah yep my bad forgot to note it uses this branch. https://github.com/hummingbird-me/kitsu-server/tree/nuck/nolt-sso Will get that updated to relocate the token in the graphql schema

lmk when you've got that branch updated and I can give it 👀 pretty quick

LGTM! Don't merge until the server side has been deployed to production yet though - I deploy the-future to production regularly with translation updates

Waiting on https://github.com/hummingbird-me/kitsu-server/pull/869

Do we want to set up the "remote Login URL" thing? nolt.io/help/single-sign-on

We probably want to do this. The canny integration has started getting spammed by non-Kitsu accounts over the past year which I've had to delete.