Skip to content

Commit b583229

Browse files
committed
chore: remove outdated comments
- jetbrains already fixed the issues related to info and cancel pop-ups shown only on the main screen
1 parent 0ece7ca commit b583229

File tree

1 file changed

+0
-15
lines changed

1 file changed

+0
-15
lines changed

src/main/kotlin/com/coder/toolbox/CoderRemoteEnvironment.kt

-15
Original file line numberDiff line numberDiff line change
@@ -125,23 +125,8 @@ class CoderRemoteEnvironment(
125125
}
126126
}
127127

128-
/**
129-
* Immediately send the state to the listener and store for updates.
130-
*/
131-
// override fun addStateListener(consumer: EnvironmentStateConsumer): Boolean {
132-
// // TODO@JB: It would be ideal if we could have the workspace state and
133-
// // the connected state listed separately, since right now the
134-
// // connected state can mask the workspace state.
135-
// // TODO@JB: You can still press connect if the environment is
136-
// // unreachable. Is that expected?
137-
// consumer.consume(status.toRemoteEnvironmentState(serviceLocator))
138-
// return super.addStateListener(consumer)
139-
// }
140-
141128
override fun onDelete() {
142129
context.cs.launch {
143-
// TODO info and cancel pop-ups only appear on the main page where all environments are listed.
144-
// However, #showSnackbar works on other pages. Until JetBrains fixes this issue we are going to use the snackbar
145130
val shouldDelete = if (wsRawStatus.canStop()) {
146131
context.ui.showOkCancelPopup(
147132
context.i18n.ptrl("Delete running workspace?"),

0 commit comments

Comments
 (0)