Follow-up from "Feat/implement germplasm result page"
The following discussions from !46 (merged) should be addressed:
-
@cyril.pommier started a discussion: (+1 comment) The name of this class seems odd to me. Search criterias should be rather generic, not a search criteria by method. I assume that there is no GermplasmGETSearchCriteria ? What does it add on top of GermplasmPOSTSearchCriteria ? FaidareGermplasmSearchCriteria is strange too. DetailGermplasmSearchCriteria instead ?
-
@cyril.pommier started a discussion: Redundant with the new method, therefore deprecated no ?
-
@cyril.pommier started a discussion: Redundant with the new method rigth below that one no ? Shouldn't it be deprecated / removed ?
-
@cyril.pommier started a discussion: (+1 comment) comment to remove
-
@cyril.pommier started a discussion: kill