-
Notifications
You must be signed in to change notification settings - Fork 172
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: Remove Handles from Graph #6040
Conversation
✅ code4rena.com deploy preview ready
|
Just a Quick update: I have a netlify function that pulls in all the data from _data/handles and returns an array of handles and I started replacing the call for judge-application.js . There are 4 places that I have seen that will need to be updated.
Edit: the handle query that was being used in the judge-application was not being used. It was being passed down to widgets => widget, but the handle options are not being applied to anything. The handle that is used for the judgeApplication function is being grabbed from the current logged in user. So I remove the logic from there. |
… removing hande calls, workin on register
@joshuashort I need to take a break on this, but I will get back to it this evening. I am working on the register.tsx. But I did add my changes here so you can see the progress here. |
have you seen the existing |
Yes, I did see that and I plan to use the |
Nice -- thanks! (and good find on the removable judge-application use!) |
…ub.com/code-423n4/code423n4.com into 6039-feature-remove-handles-from-graph
…ub.com/code-423n4/code423n4.com into 6039-feature-remove-handles-from-graph
…get users and get teams in user utils
@joshuashort I added that file in here, could you review these changes and then could we meet to talk about the next phase of these changes? |
Adding @captainmangoC4 as an "assignee" here too (whatever that means exactly wrt to a PR?) just since it's possible they might be revisiting a lot of things related to handles during some upcoming function migration work. |
Closing stale PR |
No description provided.