Skip to content

Add Input Validation and Type Safety Improvements - #81

Open
clcl777 wants to merge 4 commits into
AWeirdDev:v2from
clcl777:validation
Open

clcl777 wants to merge 4 commits into
AWeirdDev:v2from
clcl777:validation

Conversation

@clcl777

@clcl777 clcl777 commented Oct 4, 2025

Copy link
Copy Markdown

This PR implements simple input validation and type safety improvements !

  • Date validation: Validation to ensure dates are in correct YYYY-MM-DD format and not in the past
  • Trip type validation: Validation for trip types against allowed values (round-trip, one-way, multi-city)
  • Seat type validation: Validation for seat types against allowed values (economy, premium-economy, business, first)
  • Flight data validation: Check to ensure round-trip bookings have at least 2 flight segments
  • Fetch mode validation: Validation for fetch modes against allowed values
  • Data source validation: Validation for data source parameters

@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. enhancement New feature or request labels Oct 4, 2025
@abhinavm24

Copy link
Copy Markdown

Hey @clcl777 can you please look at https://github.com/AWeirdDev/flights/pull/82/files
Feel free to port your validations to my pr

@abhinavm24

Copy link
Copy Markdown

also, you have to build on dev branch instead of main

@clcl777

clcl777 commented Oct 5, 2025

Copy link
Copy Markdown
Author

Okay, I see
Plz wait a moment

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants