feat: add 'Drill from Position' functionality to the analysis screen#186
feat: add 'Drill from Position' functionality to the analysis screen#186maddavid123 wants to merge 2 commits into
Conversation
|
Someone is attempting to deploy a commit to the Maia Platform Team on Vercel. A member of the Team first needs to authorize it. |
|
Let me know if you have the time to complete implementing this; otherwise I'll implement it in my next opening page update |
|
Alright, I think this is ready for review. Open issues that I'm aware of: Also, I didn't check how this is viewed on mobile. |
|
Closing due to staleness. I imagine the repo has advanced significantly since this PR was opened, and I have no plans on updating this PR. |
Draft implementation of #185 - incomplete and possibly buggy.
Currently loads the PGN and FEN of the position in the analysis mode into the drill tool.
Current outstanding problems:
My understanding of typescript/javascript and by extension React/Next.js is very limited (My work usually is embedded C or CLI tasks in C#), so I may have missed better ways of achieving the similar behaviour or trusted the IDE a bit too much where it's unnecessary.
It's late where I am, so will follow up with any feedback/discussion points tomorrow.