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

editor suggested upgrades #196

Open
kouloumos opened this issue Oct 27, 2023 · 2 comments
Open

editor suggested upgrades #196

kouloumos opened this issue Oct 27, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@kouloumos
Copy link
Member

I observed two instances of weird behavior with the state of the data in the Editor:

  1. When I click "Restore Original" I would expect every field to go back to its original content, but this is not the case. Categories and Directory are not resetting.
  2. If I make a change to a field and then leave the Editor, although the change will be saved (there is auto-save on every metadata change) the very first time that I will come back to the Editor, I will see the previous state (the one before the change). If I refresh the page, I will get the correct up-to-date state.
@kouloumos kouloumos added bug Something isn't working and removed bug Something isn't working labels Oct 27, 2023
@adamjonas adamjonas changed the title The state of the Editor is not reliable editor suggested upgrades Nov 9, 2023
@Extheoisah Extheoisah added the bug Something isn't working label Dec 13, 2023
@BalogunofAfrica
Copy link
Collaborator

BalogunofAfrica commented Dec 18, 2023

Hi @kouloumos

  1. For directory, there is no "original" state we can restore to. The closest we can do is restore to the last saved state.
  2. I tried to reproduce but couldn't. Can you walk me through how you got this?

Here's a draft for the fixes: #238

@kouloumos
Copy link
Member Author

This is still an issue, although it's kind of different now that we are using GitHub as the source of truth for the state of the transcript/edit.

Update on observations

  1. When I click "Restore Original" categories are still not resetting. Directories has since been removed.
  2. If I make a change to a field and then leave the Editor, although the change will be saved (there is auto-save on every metadata change) the very first time that I will come back to the Editor, I will see the previous state (the one before the change). If I refresh the page, I will get the correct up-to-date state.

This is still an issue, I think it has to do with concurrency. It only occurs if you change a metadata field (e.g add/remove speaker) and then you quickly refresh the page. I think we can ignore it for now.

  1. When I click "Restore Original" , everything resets (except of the categories), but the change is not actually reflected on the commits on the review-branch - meaning that the change back to the original state is not saved automatically after "Restore Original". Because all the metadata changes are saved automatically, not saving automatically this change to the metadata (after resetting) doesn't feel intuitive based on current functionality. On the other hand, Saves to the transcript are not saved automatically (you have to press the Save button), therefore this creates a weird situation. A solution could be to have two different "Restore Original" options, one for metadata and one for the transcript.

Possible solution

I believe that all of those issues will be resolved if we change the way we are handling/showcasing-to-the-user the current state of Saved Changes. But as we are already thinking about a different editing experience (#280) it doesn't make sense to work on this right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants