Skip to content

Commit

Permalink
Review comments
Browse files Browse the repository at this point in the history
- remove FDS_ID from contact
- make holding mandatory on position
  • Loading branch information
rikoe committed Feb 13, 2020
1 parent 6006d0c commit 17cec02
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 6 deletions.
3 changes: 1 addition & 2 deletions src/context/schemas/contact.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,7 @@
"id": {
"type": "object",
"properties": {
"email": { "type": "string" },
"FDS_ID": { "type": "string" }
"email": { "type": "string" }
}
}
},
Expand Down
2 changes: 1 addition & 1 deletion src/context/schemas/position.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,5 @@
"instrument": { "$ref": "instrument.schema#/" },
"holding": { "type": "number", "minimum": 0 }
},
"required": ["instrument"]
"required": ["instrument","holding"]
}
3 changes: 1 addition & 2 deletions website/static/schemas/next/contact.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,7 @@
"id": {
"type": "object",
"properties": {
"email": { "type": "string" },
"FDS_ID": { "type": "string" }
"email": { "type": "string" }
}
}
},
Expand Down
2 changes: 1 addition & 1 deletion website/static/schemas/next/position.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,5 @@
"instrument": { "$ref": "instrument.schema#/" },
"holding": { "type": "number", "minimum": 0 }
},
"required": ["instrument"]
"required": ["instrument","holding"]
}

0 comments on commit 17cec02

Please sign in to comment.