Skip to content
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

feat: Add types for experiment apis in superposition_types #423

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

Conversation

ayushjain17
Copy link
Collaborator

@ayushjain17 ayushjain17 commented Feb 19, 2025

  1. Moved experimentation API types to superposition_types and used it in experimentation_platform
  2. Added TextToSql, TextFromSql and TextFromSqlNoValidation derive macros for db data conversion
  3. Introduced ChangeReason, Description and TrafficPercentage newtypes
  4. Introduced DisableDBValidation trait to allow legacy data migration

@ayushjain17 ayushjain17 requested a review from a team as a code owner February 19, 2025 09:42
@ayushjain17 ayushjain17 changed the title Exp/update feat: Add types for experiment apis in superposition_types Feb 19, 2025
@ayushjain17 ayushjain17 force-pushed the exp/update branch 11 times, most recently from df1a142 to 088eeb5 Compare February 21, 2025 08:26
@Datron Datron added the UI things relating to the UI label Mar 3, 2025
@ayushjain17 ayushjain17 force-pushed the exp/update branch 5 times, most recently from 09041c5 to 5ea9c9f Compare March 12, 2025 14:34
@ayushjain17 ayushjain17 force-pushed the exp/update branch 2 times, most recently from 0f207cd to b70f604 Compare March 17, 2025 06:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
UI things relating to the UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants