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

overhauled-ChanakyaGPT #372

Merged

Conversation

Nkovaturient
Copy link
Contributor

Closes: #360

  • Title : Overhaul ChanakyaGPT
  • Name: Neha Kumari
  • Idenitfy yourself: GSSOC Extd '24 || Contributor

Describe the add-ons or changes you've made 📃

  • Examined and overhaul chanakyaGPT:-
  • replaced the groq-sdk with fast, resilient and reliable Cohere-API
  • added proper logic and functionality
  • responsiveness and styling suitable on various devices

Type of change ☑️

What sort of change have you made:

    • [✅ ] Bug fix (non-breaking change which fixes an issue)
    • [ ✅ ] New feature (non-breaking change which adds functionality)
    • [✅ ] Code style update (formatting, local variables)
  • Checklist: ☑️

  • [✅ ] This PR does not contain plagiarized content.

  • [✅ ] I have performed a self-review of my own code.

  • [✅ ] I have commented my code, particularly wherever it was hard to understand.

  • [✅ ] My changes generate no new warnings.

Screenshots 📷

Screenshot (268)
Screenshot (267)

https://www.loom.com/share/1388d6914faa46178ace32c277b73379?sid=3841ba42-69f3-4e3b-87b4-5124c2c53823

Copy link

vercel bot commented Oct 29, 2024

@Nkovaturient is attempting to deploy a commit to the avdheshvarshney's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

Thank you for submitting your pull request! We'll review it as soon as possible. For further communication, join our discord server https://discord.gg/tSqtvHUJzE.

Copy link

vercel bot commented Oct 29, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
chanakya ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 31, 2024 11:58am

@Avdhesh-Varshney
Copy link
Owner

Nice work. But UI not looks good. Maintain the website theme.

@Avdhesh-Varshney Avdhesh-Varshney added the Requested Changes ⚙️ Some changes have been requested in this PR label Oct 29, 2024
@Nkovaturient
Copy link
Contributor Author

Hello @Avdhesh-Varshney ,
I hv made the desired changes. Here is the sc:-

Screenshot (269)

@Nkovaturient
Copy link
Contributor Author

Hello @Avdhesh-Varshney , I hv made the desired changes. Here is the sc:-

Screenshot (269)

Hey @Avdhesh-Varshney !
anything else u would like me to update?

Copy link
Owner

@Avdhesh-Varshney Avdhesh-Varshney left a comment

Choose a reason for hiding this comment

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

It's still not responding.
image

@Nkovaturient
Copy link
Contributor Author

It's still not responding. image

Hello @Avdhesh-Varshney ,
it certainly has to do with the apiKey config/ import or deploy changes, bcuz GPT is workin perfectly fine in my codebase--
Demo of today-

https://www.loom.com/share/ece6321d8f0f4e858828a606987fa288?sid=f207aef7-3dba-4025-9002-740fed236f51

Copy link
Owner

@Avdhesh-Varshney Avdhesh-Varshney left a comment

Choose a reason for hiding this comment

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

Ok, I'll fix it

@Avdhesh-Varshney Avdhesh-Varshney merged commit 19bdbaa into Avdhesh-Varshney:main Oct 31, 2024
3 checks passed
@Avdhesh-Varshney Avdhesh-Varshney added enhancement 🧑‍💻 New feature or request level2 Intermediate level issues/pull requests Approved ✔️ This PR is approved by the PR or, Mentors gssoc-ext gssoc extended 2024 hacktoberfest-accepted Hacktoberfest 2024 and removed Requested Changes ⚙️ Some changes have been requested in this PR labels Oct 31, 2024
@Nkovaturient
Copy link
Contributor Author

Nkovaturient commented Oct 31, 2024

Ok, I'll fix it

Hey, @Avdhesh-Varshney
I have fixed it

@Nkovaturient
Copy link
Contributor Author

Ok, I'll fix it

Basically it was package dependency issue
I have updated and resolved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved ✔️ This PR is approved by the PR or, Mentors enhancement 🧑‍💻 New feature or request gssoc-ext gssoc extended 2024 hacktoberfest-accepted Hacktoberfest 2024 level2 Intermediate level issues/pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛: ChanakyaGPT || Not Responding
2 participants