Skip to content
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

Delete earlier moves#12986 WIP #14486

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ross39
Copy link
Contributor

@ross39 ross39 commented Jan 16, 2024

I would like to get some feedback on adding a new button delete earlier moves #12986. It currently works(I think :) ) but the problem is that when used outside the analysis board, when the moves are deleted and you go to create the study from the game, the side view shows the supposed to be deleted moves. Any ideas on why this happens? I thought maybe it was a socket issue so I modified analyse/socket.ts but that did not fix the issue. Thanks

@lenguyenthanh
Copy link
Member

This will required changes in lila-ws (for support new socket command) and lila backend for update study to database.

@lenguyenthanh lenguyenthanh marked this pull request as draft January 17, 2024 02:05
@ross39
Copy link
Contributor Author

ross39 commented Jan 21, 2024

pushing what I have done so far. I am confused on one thing as I am trying to figure out how deleteNode works as its quite similar to delete earlier moves just instead of deleting from the current node down the tree I am trying to delete from the current node to the parent(root).

I have a question. If I open a recently played game and click 'Delete From Here' and then add the game to the study why does the full pgn still appear in the study?. To make my question more clear

  1. Go to recently played game (https://lichess.org/baz6CzB0/black)
  2. Click 'Delete From Here' on any move. This correctly deletes the moves from the view
  3. Click menu and then click study and add the game to a study
  4. The view shows the supposed to be deleted moves.

The reason I ask this is because 'Delete Earlier Moves' deletes the moves in the view but the moves are still there when I create a study from the game as described above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants