Wormhole code removal #103

Closed
opened 1 year ago by onryo · 4 comments
onryo commented 1 year ago
Collaborator

Since we now have a standalone Android wallet we no longer need to support the code for wormhole.

Since we now have a standalone Android wallet we no longer need to support the code for wormhole.
onryo added the
low priority
label 1 year ago
fekt commented 1 year ago
Collaborator

The easiest thing to do for now to avoid any confusion with mobile app would be to remove the Apps menu option and Ctrl+M shortcut. Any other code can be removed later. I can do this today unless someone else has started on it. Let me know.

The easiest thing to do for now to avoid any confusion with mobile app would be to remove the Apps menu option and Ctrl+M shortcut. Any other code can be removed later. I can do this today unless someone else has started on it. Let me know.
Poster
Collaborator

The easiest thing to do for now to avoid any confusion with mobile app would be to remove the Apps menu option and Ctrl+M shortcut. Any other code can be removed later. I can do this today unless someone else has started on it. Let me know.

I was thinking the same. 👍

> The easiest thing to do for now to avoid any confusion with mobile app would be to remove the Apps menu option and Ctrl+M shortcut. Any other code can be removed later. I can do this today unless someone else has started on it. Let me know. I was thinking the same. 👍
fekt commented 1 year ago
Collaborator

Apps menu option is removed now. I didn't want to get into deleting files and other code yet. Most of the code is in websockets.cpp but I wasn't sure if that's all only used for the old mobile app or not.

Apps menu option is removed now. I didn't want to get into deleting files and other code yet. Most of the code is in `websockets.cpp` but I wasn't sure if that's all only used for the old mobile app or not.
duke commented 1 year ago
Owner

done

done
duke closed this issue 1 year ago
Sign in to join this conversation.
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.