8000 https://github.com/FoloUp/FoloUp/issues/3 by aditya-qapitol · Pull Request #5 · FoloUp/FoloUp · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

https://github.com/FoloUp/FoloUp/issues/3 #5

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
Merged
merged 3 commits into from
Feb 17, 2025
Merged

https://github.com/FoloUp/FoloUp/issues/3 #5

merged 3 commits into from
Feb 17, 2025

Conversation

aditya-qapitol
Copy link
Contributor

This contains the updated V2 API of Retell which resolves the issue of the web call agent not starting.

@aditya-qapitol aditya-qapitol mentioned this pull request Feb 14, 2025
Closed
SuveenE
SuveenE requested changes Feb 14, 2025
View reviewed changes
Copy link
Contributor
@SuveenE SuveenE left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the changes Aditya! Requested a minor change. Also please add a note on how you tested it.

@aditya-qapitol
Copy link
Contributor Author

Removed the unused variable.
Tested as follows:

  1. Executed yarn dev
  2. Logged in using Google authentication
  3. Add/Create agents if not present
  4. Click on create interview
  5. Enter name
  6. Select agent
  7. Provide file as input
  8. Provide duration & number of questions
  9. Click on generate questions
  10. Select the questions and submit
  11. Click on the URL share present in the corner of the icon of the newly created interview.
  12. Paste the URL in a new Browser window (Opened in Guest mode)
  13. Enter the email & name
  14. Click on start Interview
  15. The agent is now able to start communicating/Talking - Works as expected now without closing

@aditya-qapitol
Copy link
Contributor Author

Fixed this commit

@SuveenE
Copy link
Contributor
SuveenE commented Feb 17, 2025

@aditya-qapitol I think you did not mean to close the PR. Feel free to raise a new one.

SuveenE-TF

This comment was marked as off-topic.

Sign in to view

@SuveenE
Copy link
Contributor
SuveenE commented Feb 17, 2025 •
edited
Loading

There's a small issue with the styling. Can you run npx prettier --write . in the home directory and push the changes please? @aditya-qapitol
Really appreciate the help and effort!

@aditya-qapitol
Copy link
Contributor Author

There's a small issue with the styling. Can you run npx prettier --write . in the home directory and push the changes please? @aditya-qapitol Really appreciate the help and effort!

Resolved it!

@SuveenE SuveenE merged commit 3e2e6de into FoloUp:main Feb 17, 2025
1 check passed
@aditya-qapitol aditya-qapitol mentioned this pull request Feb 17, 2025
Open
DreamKeeper41012 added a commit to DreamKeeper41012/FoloUp that referenced this pull request Mar 27, 2025
@DreamKeeper41012
[REMOVED] FoloUp/FoloUp#5
41fc600
DreamKeeper41012 added a commit to DreamKeeper41012/FoloUp that referenced this pull request Mar 27, 2025
@DreamKeeper41012
[UPDATED] FoloUp/FoloUp#5 (comment)
2b2ddb3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SuveenE SuveenE SuveenE requested changes

@SuveenE-TF SuveenE-TF SuveenE-TF approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@aditya-qapitol @SuveenE @SuveenE-TF
0