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

Chat with GitHub Frontend UI #170

Merged
merged 3 commits into from
Mar 29, 2024
Merged

Chat with GitHub Frontend UI #170

merged 3 commits into from
Mar 29, 2024

Conversation

bryannho
Copy link
Contributor

@bryannho bryannho commented Mar 29, 2024

Changes

  • Updates chat-with-github-api example to include Solara frontend
  • Reorganizes app into api/ and client/ folders
  • Updates Dockerfile to deploy all processes in one container

Closes #148


📚 Documentation preview 📚: https://ploomber-doc--170.org.readthedocs.build/en/170/

@bryannho bryannho marked this pull request as ready for review March 29, 2024 07:24
@bryannho bryannho requested a review from edublancas as a code owner March 29, 2024 07:25
# with sl.Column(align="start", classes=["plug"]):
# with sl.HBox(align_items="center"):
# sl.Image(f"static/system-logo.png", classes=["logo"])
# sl.HTML(tag="a", unsafe_innerHTML="Built by Ploomber", attributes={"href": "https://ploomber.io", "target": "_blank"}, classes=["link"])
Copy link
Contributor

Choose a reason for hiding this comment

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

ensure you remove commented code

Copy link
Contributor Author

Choose a reason for hiding this comment

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

humanize
solara==1.29.1
openai
Copy link
Contributor

Choose a reason for hiding this comment

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

provide specific versions so the app doesn't break with upgrades

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@bryannho bryannho requested a review from edublancas March 29, 2024 14:47
@edublancas edublancas merged commit c057c04 into main Mar 29, 2024
1 check passed
@edublancas edublancas deleted the chat-with-gh-frontend branch March 29, 2024 15:44
@edublancas
Copy link
Contributor

@bryannho are we linking from the blog post to the example? if not, please add a link

@bryannho
Copy link
Contributor Author

@bryannho are we linking from the blog post to the example? if not, please add a link

@edublancas the blog post links to the example code but not the deployed app. do you want me to add a link to the deployed app?

@edublancas
Copy link
Contributor

no, it's ok. i meant a link to the example code

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.

chat with github repository
2 participants