Skip to content

Make ‚Update all Screenshots’ a boolean workflow option#839

Merged
carstingaxion merged 1 commit intoGatherPress:mainfrom
carstingaxion:fix/screenshot-workflow-trigger-should-be-boolean
Sep 3, 2024
Merged

Make ‚Update all Screenshots’ a boolean workflow option#839
carstingaxion merged 1 commit intoGatherPress:mainfrom
carstingaxion:fix/screenshot-workflow-trigger-should-be-boolean

Conversation

@carstingaxion
Copy link
Copy Markdown
Collaborator

@carstingaxion carstingaxion commented Sep 3, 2024

image

Description of the Change

This is a small follow-up on #838 to make the selectable option of the workflow a Boolean. Having this as a string makes no sense.

Closes #

How to test the Change

Changelog Entry

Fixed - Workflow input to be a bool option.

Credits

Props @carstingaxion

Checklist:

  • I agree to follow this project's Code of Conduct.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests pass.

@carstingaxion carstingaxion marked this pull request as ready for review September 3, 2024 04:47
@carstingaxion carstingaxion changed the title Try to make a bool out of the option Make ‚Update all Screenshots’ a boolean workflow option Sep 3, 2024
@carstingaxion carstingaxion merged commit 5ae505b into GatherPress:main Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant