-
Notifications
You must be signed in to change notification settings - Fork 97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TASK-1562 Pedigree graph with a three-generation family fails to render properly in Family browser detailed tab #2253
Conversation
…ASK-3898, #TASK-1562
…-3898, #TASK-1562
…a model, #TASK-4000, #TASK-1562
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure which field name I would use to contain the base 64 pedigree graph image, but I would certainly not call it b64PedigreeGraph. If everyone's happy with that, we can go ahead.
@@ -153,6 +153,9 @@ public class Family extends Annotable { | |||
description = FieldConstants.FAMILY_ROLES) | |||
private Map<String, Map<String, FamiliarRelationship>> roles; | |||
|
|||
@DataField(id = "b64PedigreeGraph", description = FieldConstants.FAMILY_PEDIGREE_GRAPH) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it really going to be called that way?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, finally, this will be an object :
"pedigreeGraph" : {"base64":"..", "json":{...}}
…n addition to the Base64 format, pedigree graph is saved in JSON format too, #TASK-4000, #TASK-1562
…n the Family data model, #TASK-4000, #TASK-1562
|
||
import static org.junit.Assert.*; | ||
|
||
public class FamilyAnalysisTest extends GenericTest { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reminder : We'll need to add the @Category
to this test once the categories are merged
…s of a given study, #TASK-4067, #TASK-1562
Implement pedigree graph.