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.

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

Update README.md for insurance agent #347

Closed
wants to merge 2 commits into from
Closed

Update README.md for insurance agent #347

wants to merge 2 commits into from

Conversation

cotran2
Copy link
Contributor

@cotran2 cotran2 commented Dec 20, 2024

No description provided.

@cotran2 cotran2 requested a review from salmanap December 20, 2024 23:36
```yaml
docker compose --profile monitoring up
```
1. Navigate to http://localhost:3000/ to open grafana UI (use admin/grafana as credentials)
1. Navigate to http://localhost:3000/ to op![Uploading Screenshot 2024-12-20 at 3.32.49 PM.png…]()
Copy link
Contributor

Choose a reason for hiding this comment

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

This isn't rendering correctly. Please fix this image.

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, I think we should fix/remove the tests for the demos that are failing (e2e demo tests) in this branch. Much easier to clean up in a single go.

Copy link
Contributor

@adilhafeez adilhafeez left a comment

Choose a reason for hiding this comment

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

@cotran2 were you able to resolve Salman's comments in the review?

@salmanap
Copy link
Contributor

salmanap commented Feb 7, 2025

I am removing the insurance agent. It doesn't add a whole lot of value. That PR will be up and coming shortly.

@salmanap salmanap closed this Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants