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] Added static map images #89

Draft
wants to merge 19 commits into
base: main
Choose a base branch
from
Draft

[Feature] Added static map images #89

wants to merge 19 commits into from

Conversation

mayurvir
Copy link
Contributor

@mayurvir mayurvir commented Apr 19, 2024 •
edited
Loading

Description

This PR adds static map image to the response send to whatsapp when:

  • User asks for routes between two places
  • User selects a route. It also shares a navigation url in this case
  • User searches along a selected route

Features covered

Changes made

  • Added media urls to Agent.js
  • Added map url for search along a polygon in Ai.js
  • Added navigation url, static image url and markers in MapService.js
  • Removed session clearance on server start
  • Updated test cases

Screenshots/recordings

static.image.mp4

Testing

Try it out yourself by interacting with the agent directly:
https://wa.me/+919335513882

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
- modified getRoutes function
b950728 
- Added one function to get response or perform an action using function calling
@mayurvir
updated sessions for the new method and tested workign search and sel…
97f52b6 
…ect actions
@mayurvir
Added schemas for init and confirm, added retries and fixes
c3643a7
@mayurvir
fixed test cases
d20ea81
@mayurvir
Tested ene to en primary booking flow
dd23470
@mayurvir
- Added searching along the route polyline
bdd1323
@mayurvir
Added raw response if raw_yn is sent
c37e9bc
@mayurvir
Added whatsapp message
73b62a4
@mayurvir
Cleanup old structure code and removed unused files and directories
095a8ef
@mayurvir
Removed unsed data files under tests
d6b51b7
@mayurvir
made fixes for failing test case
fa71670
@mayurvir
Added route image in the response while getting routes
f6ce817
@mayurvir
Added route image and navigation url to response of select_route call
d00c7d2
@mayurvir
Added static map image on search along the route
5110dcb
@mayurvir
restricted to failining tests
0924c69
@mayurvir
Added debug tests
722095c
@mayurvir
Skipped test cases causing flakiness
1cdff7b
@mayurvir mayurvir marked this pull request as draft April 19, 2024 07:00
@mayurvir mayurvir linked an issue Apr 19, 2024 that may be closed by this pull request
Add static map images for selected routes #83
Open
7 tasks
@mayurvir mayurvir self-assigned this Apr 19, 2024
@mayurvir mayurvir added the enhancement New feature or request label Apr 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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.

Add static map images for selected routes
1 participant
@mayurvir