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

fix(ui): disable export button if no non-default presets #6773

Merged
merged 2 commits into from
Aug 26, 2024

Conversation

maryhipp
Copy link
Collaborator

Summary

Instead of counting the number of user presets, count the number of non-default presets so that other types are included.

Related Issues / Discussions

QA Instructions

Merge Plan

Checklist

  • The PR has a short but descriptive title, suitable for a changelog
  • Tests added / updated (if applicable)
  • Documentation added / updated (if applicable)

@maryhipp maryhipp changed the title disable export button if no non-default presets fix(ui): disable export button if no non-default presets Aug 23, 2024
@github-actions github-actions bot added the frontend PRs that change frontend files label Aug 23, 2024
@maryhipp maryhipp enabled auto-merge (rebase) August 26, 2024 13:21
@maryhipp maryhipp merged commit 4e294f9 into main Aug 26, 2024
14 checks passed
@maryhipp maryhipp deleted the maryhipp/fix-shared-preset-export branch August 26, 2024 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend PRs that change frontend files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants