Change to true/false to yes/no in Auto Capture Select Text

[michaelmichael]

Description
Ref #3697

This modifies the text in the Auto Capture dropdown menu to make the verbiage more consistent. It checks whether Spree::Config.auto_capture is set to true or false. If it is set to true, the primary dropdown will read "Use App Default (Yes)". If it is set to "false"—or if it is not set (the default config state)—it will read "Use App Default (No)".

Spree::Config.auto_capture = true:

Spree::Config.auto_capture = false:

Checklist:

• I have followed Pull Request guidelines
• I have added a detailed description into each commit message
• I have updated Guides and README accordingly to this change (if needed)
• I have added tests to cover this change (if needed)
• I have attached screenshots to this PR for visual changes (if needed)

[jarednorman]

Looks good, though I'd be open to an attempt to make this a bit easier to read. There's a lot going on on this line.

[aldesantis]

Looks good and I agree with @jarednorman. @michaelmichael perhaps you can just put the arguments on different lines? I think that alone would make them easier to parse. 🙏

[kennyadsl]

@michaelmichael let us know if you have time to make the suggested improvements, even in another PR so we can go ahead merging this. Thanks a lot for your contributions!