Skip to content

Fix the chatbot demo #169

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

Closed
wants to merge 1 commit into from
Closed

Fix the chatbot demo #169

wants to merge 1 commit into from

Conversation

fabpot
Copy link
Member

@fabpot fabpot commented Jul 20, 2025

Q A
Bug fix? yes
New feature? no
Docs? no
Issues
License MIT

Updated the code to the current names.

@fabpot fabpot force-pushed the demo-chatbot-fix branch from e6c8066 to 15e93ea Compare July 20, 2025 07:57
@chr-hertel
Copy link
Member

That shouldn't be needed anymore, we merged #154 already

@fabpot
Copy link
Member Author

fabpot commented Jul 20, 2025

That's because there is a composer.lock file in the demo directory. I suggest that we remove it until we have a first release. WDYT?

@chr-hertel
Copy link
Member

Ah, yes, I feel like that demo doesn't play well right now with the mono repo. removing the composer.lock might help 👍

@chr-hertel
Copy link
Member

And maybe switching to Symfony CLI as a next step for the PHP runtime

@fabpot
Copy link
Member Author

fabpot commented Jul 20, 2025

And maybe switching to Symfony CLI as a next step for the PHP runtime

I can work on it :)

What about using PostgreSQL for the vector DB? That would allow for a nice and simple way for developers to get started.

@chr-hertel
Copy link
Member

Postgres would easily integrate with standard DB tooling, yea, i like that - and since we have that bridge now, yes please :)

@fabpot
Copy link
Member Author

fabpot commented Jul 20, 2025

See #170 for a better fix.

@fabpot fabpot closed this Jul 20, 2025
chr-hertel added a commit that referenced this pull request Jul 20, 2025
This PR was merged into the main branch.

Discussion
----------

Remove composer.lock until we have a first release

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Docs?         | no <!-- required for new features -->
| Issues        | Closes #169
| License       | MIT

Until we have a release, it's better to always get the latest version of the code.

Commits
-------

6b4ed7f Remove composer.lock until we have a first release
@chr-hertel chr-hertel added the Demo Issues & PRs about the demo application label Jul 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Demo Issues & PRs about the demo application
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants