-
Notifications
You must be signed in to change notification settings - Fork 85
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
Add StockWise Chatbot Integration for Stock Price Prediction #224
Conversation
Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ensure the PR matches the requirements mentioned in the Contribution guide. The maintainer might get in touch to enusre quality. Thanks for your time
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@BigBang001 please sync your fork with the latest updates:
@rohitinu6 I have synced the fork with the latest updates. Now you can check and verify. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@BigBang001 Please link the issue using keywords. (fixes/closes..) You can find the detailed info about the same here:
https://docs.github.com/en/issues/tracking-your-work-with-issues/using-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword
If needed, please check out merged PRs for reference.
Thanks & regards
@Mayureshd-18 I have linked the PR with the respective issue. Now you can check and verify. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the "Id" in stock.html should be mentioned in a .env file instead of directly into the html file.
@rohitinu6 What is your opinion on this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
🎉🎉 Thank you for your contribution! Your PR #224 has been merged! 🎉🎉 |
Pull Request Description:
This PR integrates the StockWise chatbot into the Stock Price Prediction project, enhancing user experience by providing real-time insights, guidance, and stock information.
Fixes #216
Key Features:
Stock Predictions and Market Insights
Performance Indicators and Influencing Factors
User Guidance for App Navigation and Prediction Understanding
Screenshots of Integration
Chatbot Icon on Homepage
131604](https://github.com/user-attachments/assets/3fe415b9-f7c4-4daa-93e0-38126de28d67)
Chat Interface with Stock Prediction Queries