Behavior change in 0.8.19 with `default`, `required`, and `skip_serializing_if` about schemars

[GREsau]

This is an interesting case, and frankly I'm not 100% sure what the "correct" behaviour here is with this complicated combination of default/required/skip_serializing_if/Option<>.

In any case, I think this does what you want, and works in all versions:

#[derive(JsonSchema)]
pub struct WithoutNullable {
    #[schemars(default, with = "String")]
    #[serde(skip_serializing_if = "Option::is_none")]
    pub value: Option<String>,
}

Does that solve your problem?

[ahl]

Sure! Or we can hand roll a JsonSchema impl. I should have been clearer: this behavior changed in a patch release. If that's intended / ok: please close this issue. If it's unintended then perhaps reinstate the behavior and/or change it in a minor release?

[GREsau]

Thanks for reporting this, I've now released 0.8.20 which restores the previous behaviour