-
Notifications
You must be signed in to change notification settings - Fork 30
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
[WIP] feat: agent rework #243
base: main
Are you sure you want to change the base?
Conversation
2561fcc
to
6734492
Compare
What's missing here exactly? |
Caught up with your changes: #242 |
My current thoughts are.
|
Seems like the last two PRs dealt with the latter, correct me if I'm wrong and something is missing. |
@avdb13 I think it would be better to do the agent rework first, but it is a bit tedious and involves some processing specific to atproto, and I think it would be better for me to do it. As for the OAuthSession part, only the |
I'm gonna wait for this PR to be completed as sebadob/rauthy#644 requires the Agent to be available for us to fetch the email associated with the user's ATProto account, prompting for one if not present. |
@sugyan it's been a month since the last commit, I really wanna take on implementing the remaining changes so could you please share instructions? Sorry for my impatience in advance. |
@avdb13 I am very sorry for the halt in progress. |
I knew something was wrong but I was hesitant to ask about it as this project is driven by voluntary contribution after all. Best of luck and merry christmas! |
No description provided.