Added a checkbox to transmit location information or not. #48
Conversation
…le to not transmit any data at all, temp - incase of doing things in the wild, etc.
| @@ -89,14 +89,16 @@ public void submitToApi() { | |||
|
|
|||
| Map<String, Object> updates = new HashMap<>(); | |||
| updates.put("name", playerName); | |||
| if (locationOption()) { | |||
There was a problem hiding this comment.
I am looking to accept this, but I want to also send the option selection to the api so it can disable sending the data from there. I would also like to indicate on the website that a player has disabled this. I will try to add this in a few days.
There was a problem hiding this comment.
@christoabrown Have you been able to add this to the API? <3
| return true; | ||
| } | ||
|
|
||
| @ConfigItem( |
There was a problem hiding this comment.
Not sure if this makes sense to have since all the plugin does is send data so a user could disable it in runelite to get the same effect.
There was a problem hiding this comment.
The in-game activity one makes sense for this, as disabling the in-game activity option is essentially disabling the plugin without disabling it. However, the disable location, just prevents your team from seeing your location but allows for everything else to appear.
…e activity transmission to advise that it also includes location.
…No different than just disabling your plugin as a whole. However, left location as disabling location while keeping the app active is helpful in a lot of ways.
Created ability to prevent the location information from being sent over to the backend.