Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ignore userFraction if status is draft #140

Merged
merged 2 commits into from
Sep 12, 2022
Merged

Conversation

boswelja
Copy link
Collaborator

No description provided.

@boswelja boswelja merged commit 5427bcb into master Sep 12, 2022
@boswelja boswelja deleted the fix/draft-publishing branch September 12, 2022 08:23
@Swisyn
Copy link
Contributor

Swisyn commented Sep 12, 2022

@boswelja Thank you for fixing the issue! When will this fix be released? We have a release today, and this is also blocking us to release our app via this action. That would be super nice to know the next release date. 🙏

@boswelja
Copy link
Collaborator Author

I don't have a planned date, I'm currently in the process of cleaning up code with the goal of making it testable so I don't think it will be soon. You should be able to roll back the action by one version to fix the issue in the meantime

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.

2 participants