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.

Sign up for GitHub

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

Jump to bottom

[Feature] Reduce itinerary questions #56

Merged
merged 3 commits into from
Apr 5, 2024
Merged

[Feature] Reduce itinerary questions #56

merged 3 commits into from
Apr 5, 2024

Conversation

mayurvir
Copy link
Contributor

@mayurvir mayurvir commented Apr 5, 2024 •
edited
Loading

Description

This PR will reduce the number of questions asked during the itinerary planning.

Features covered

Changes made

  • Update get_ai_response_to_query prompt to ask fewer questions for itinerary planning and use information from user profile.
  • Added test cases

Screenshots/recordings

NA

Testing

Self checks

  • Code follows coding style guidelines.
  • All tests pass successfully.
  • No new linting errors or warnings.
  • Documentation is updated, if necessary.
  • Dependencies are updated, if required.
  • There are no merge conflicts.

Reviewer Suggestions

None

@mayurvir mayurvir self-assigned this Apr 5, 2024
@mayurvir mayurvir requested review from em-abee and shreyvishal April 5, 2024 05:50
@mayurvir mayurvir added the enhancement New feature or request label Apr 5, 2024
@mayurvir mayurvir merged commit 1d36589 into main Apr 5, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@em-abee em-abee Awaiting requested review from em-abee

@shreyvishal shreyvishal Awaiting requested review from shreyvishal

Assignees

@mayurvir mayurvir

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mayurvir