Add handling for JsNull case in Utils.scala to return empty string#1226
Add handling for JsNull case in Utils.scala to return empty string#1226CollinBeczak wants to merge 1 commit intomainfrom
Conversation
|
jake-low
left a comment
jake-low
left a comment
There was a problem hiding this comment.
Can you provide some more context? This seems like a weird fix for the issue. Utils.getProperties() is called in the Task creation path, so this change affects what gets stored in the database, not just what gets rendered in the frontend.
|
What i think was happening was when that code block got a null value it would convert it to string value: "null", so i thought that if i just turned it into an empty string that would be fine for now. I suppose i can experiment with having it have a null value and see how the database and frontend handles it, and then make fixes if there are any issues. |
2 participants
Resolves: maproulette/maproulette3#2822