Skip to content

BugFix: Enum support in compound keys #24

New issue

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

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

Already on GitHub? # to your account

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

abhinand-c
Copy link
Contributor

@abhinand-c abhinand-c commented Jun 12, 2023

Fixes issue #23

When using Enum Field as a key in containing a compound statement, validation fails and doesn't work (raises AssertionError: Invalid compound key definition for type). But the same works if there is no compound key, only enum key.

Changes:

  • fix(Compound Key): Add validation for EnumType, fixes Invalid compound key definition for EnumType
  • feat(Tests): Add test for compound key with Enum types
  • refactor(Compound Key): cleaner logic

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant