You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The nice way to implement this feature (with the buttons inside the tab on the right-hand panel) would have been to just open the tab fully client-side and not send any request to the server until the user pressed the button to choose between the two approaches.
Unfortunately, I implemented the tabs assuming that one tab <-> one Job object on the server.
I wrote the client in Javascript and not Typescript, and as a result, it is kind of hard to understand it and refactor it to allow tabs that are not jobs while feeling confident that the refactoring will be a success.
As a result of that, and because we are on a deadline, I have instead implemented this feature in a slightly hacky way to avoid performing that tricky refactoring.
The places that need to be rewritten include basically everything I added in the branch for #82. I will also put TODOs in some specific spots that may not be so obvious.
The text was updated successfully, but these errors were encountered:
The nice way to implement this feature (with the buttons inside the tab on the right-hand panel) would have been to just open the tab fully client-side and not send any request to the server until the user pressed the button to choose between the two approaches.
Unfortunately, I implemented the tabs assuming that one tab <-> one Job object on the server.
I wrote the client in Javascript and not Typescript, and as a result, it is kind of hard to understand it and refactor it to allow tabs that are not jobs while feeling confident that the refactoring will be a success.
As a result of that, and because we are on a deadline, I have instead implemented this feature in a slightly hacky way to avoid performing that tricky refactoring.
The places that need to be rewritten include basically everything I added in the branch for #82. I will also put TODOs in some specific spots that may not be so obvious.
The text was updated successfully, but these errors were encountered: