Skip to main content

FIX SAVE STATE OF COMMENTS

Bug Reports

Switching between old trades, say id 15 trade and id 16 trade. the comments sometimes carries over the comment of a previously viewed trade like id 15 then when i open/check another old trade id 16 it replaces the comments of id 16 with id 15 permanently. very annoying then it saves state automatically.

How does this impact your experience?
Posted by Mahmoud Mahmoud ·
Status: Planned3 comments

Comments3

  • Stanislav Voitsekhovskyi

    Team•

    Sep 28

    Dear @Mahmoud Mahmoud,

    Thank you for your feedback.

    I’ve already passed it to our development team so they can investigate it.

    For now, could you please leave the trades and their comments unchanged? This will allow our developers to check the issue directly on your project and reproduce what happened.

    If they need any additional information from you, I’ll let you know.

    • Stanislav Voitsekhovskyi

      Team•

      Oct 8

      Hi @Mahmoud Mahmoud,

      Thank you for your patience. Our developer has a follow-up question to help investigate this issue.

      Is it possible that this happens after using Ctrl+Z in the comment text editor? This could be one of the cases that causes the comment to be carried over from a previously viewed trade.

      Could you please confirm whether you use Ctrl+Z when editing trade comments?

      • Mahmoud Mahmoud

        •

        Oct 8

        It wasn’t control z, The main issue is i’ve found it to be a race condition issue, because i loaded one old trade and i quickly switch to another trade in my history it switches to the selected trade while the old state is still in memory so the newly selected trade treats it as an autosave, by carrying over all the comments and tags from the previously viewed trade. i don’t know the state management tool you’re using but make sure to dismount one viewed trade’s state before viewing the next trade that way we don’t save what was previously loaded in state.