Scouting: Fix sketchy pickup bug
Note scouts were seeing a solid pickup label but the app was submitting
sketchy pickups instead. Strategy team wants to measure solid pickups, this change
fixes the bug so that the solid pickup keyword is actually submitted.
Signed-off-by: Filip Kujawa <filip.j.kujawa@gmail.com>
Change-Id: I7578b4ddb64f176409b0e666e44675239eb25475
diff --git a/scouting/webserver/requests/debug/cli/cli_test.py b/scouting/webserver/requests/debug/cli/cli_test.py
index 4763058..2fc4d6c 100644
--- a/scouting/webserver/requests/debug/cli/cli_test.py
+++ b/scouting/webserver/requests/debug/cli/cli_test.py
@@ -188,7 +188,7 @@
"notes": "A very inspiring and useful comment",
"good_driving": True,
"bad_driving": False,
- "sketchy_pickup": False,
+ "solid_pickup": False,
"sketchy_placing": True,
"good_defense": False,
"bad_defense": False,
@@ -211,7 +211,7 @@
Notes: (string) (len=35) "A very inspiring and useful comment",
GoodDriving: (bool) true,
BadDriving: (bool) false,
- SketchyPickup: (bool) false,
+ SolidPickup: (bool) false,
SketchyPlacing: (bool) true,
GoodDefense: (bool) false,
BadDefense: (bool) false,
diff --git a/scouting/webserver/requests/messages/request_all_notes_response.fbs b/scouting/webserver/requests/messages/request_all_notes_response.fbs
index 78e662d..983ff62 100644
--- a/scouting/webserver/requests/messages/request_all_notes_response.fbs
+++ b/scouting/webserver/requests/messages/request_all_notes_response.fbs
@@ -5,7 +5,7 @@
notes:string (id: 1);
good_driving:bool (id: 2);
bad_driving:bool (id: 3);
- sketchy_pickup:bool (id: 4);
+ solid_pickup:bool (id: 4);
sketchy_placing:bool (id: 5);
good_defense:bool (id: 6);
bad_defense:bool (id: 7);
diff --git a/scouting/webserver/requests/messages/submit_notes.fbs b/scouting/webserver/requests/messages/submit_notes.fbs
index 27ed472..cee4dee 100644
--- a/scouting/webserver/requests/messages/submit_notes.fbs
+++ b/scouting/webserver/requests/messages/submit_notes.fbs
@@ -5,7 +5,7 @@
notes:string (id: 1);
good_driving:bool (id: 2);
bad_driving:bool (id: 3);
- sketchy_pickup:bool (id: 4);
+ solid_pickup:bool (id: 4);
sketchy_placing:bool (id: 5);
good_defense:bool (id: 6);
bad_defense:bool (id: 7);
diff --git a/scouting/webserver/requests/requests.go b/scouting/webserver/requests/requests.go
index 4f12a4f..a785e18 100644
--- a/scouting/webserver/requests/requests.go
+++ b/scouting/webserver/requests/requests.go
@@ -482,7 +482,7 @@
Notes: string(request.Notes()),
GoodDriving: bool(request.GoodDriving()),
BadDriving: bool(request.BadDriving()),
- SketchyPickup: bool(request.SketchyPickup()),
+ SolidPickup: bool(request.SolidPickup()),
SketchyPlacing: bool(request.SketchyPlacing()),
GoodDefense: bool(request.GoodDefense()),
BadDefense: bool(request.BadDefense()),
@@ -856,7 +856,7 @@
Notes: note.Notes,
GoodDriving: note.GoodDriving,
BadDriving: note.BadDriving,
- SketchyPickup: note.SketchyPickup,
+ SolidPickup: note.SolidPickup,
SketchyPlacing: note.SketchyPlacing,
GoodDefense: note.GoodDefense,
BadDefense: note.BadDefense,
diff --git a/scouting/webserver/requests/requests_test.go b/scouting/webserver/requests/requests_test.go
index d2d5acc..7aa670d 100644
--- a/scouting/webserver/requests/requests_test.go
+++ b/scouting/webserver/requests/requests_test.go
@@ -565,7 +565,7 @@
Notes: "Notes",
GoodDriving: true,
BadDriving: false,
- SketchyPickup: true,
+ SolidPickup: true,
SketchyPlacing: false,
GoodDefense: true,
BadDefense: false,
@@ -583,7 +583,7 @@
Notes: "Notes",
GoodDriving: true,
BadDriving: false,
- SketchyPickup: true,
+ SolidPickup: true,
SketchyPlacing: false,
GoodDefense: true,
BadDefense: false,
@@ -603,7 +603,7 @@
Notes: "Notes",
GoodDriving: true,
BadDriving: false,
- SketchyPickup: true,
+ SolidPickup: true,
SketchyPlacing: false,
GoodDefense: true,
BadDefense: false,
@@ -831,7 +831,7 @@
Notes: "Notes",
GoodDriving: true,
BadDriving: false,
- SketchyPickup: true,
+ SolidPickup: true,
SketchyPlacing: false,
GoodDefense: true,
BadDefense: false,
@@ -842,7 +842,7 @@
Notes: "More Notes",
GoodDriving: false,
BadDriving: false,
- SketchyPickup: false,
+ SolidPickup: false,
SketchyPlacing: true,
GoodDefense: false,
BadDefense: true,
@@ -870,7 +870,7 @@
Notes: "Notes",
GoodDriving: true,
BadDriving: false,
- SketchyPickup: true,
+ SolidPickup: true,
SketchyPlacing: false,
GoodDefense: true,
BadDefense: false,
@@ -881,7 +881,7 @@
Notes: "More Notes",
GoodDriving: false,
BadDriving: false,
- SketchyPickup: false,
+ SolidPickup: false,
SketchyPlacing: true,
GoodDefense: false,
BadDefense: true,