not sure this is required given your impl? but checkout the next comment we can probably streamline