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

Corrected Environment Variable for Alan AI API Key in Alan.js Componen #447

Merged
merged 3 commits into from
Aug 10, 2023
Merged

Conversation

Nishitbaria
Copy link
Contributor

Fixes Issue

#439

Pull Request Description:

This pull request addresses an issue related to the environment variable setup for the Alan AI API key within the Alan.js component. The problem caused incorrect or missing API key usage, leading to unexpected behavior and failed interactions with the Alan AI platform.

Bug Description:

Previously, the environment variable responsible for storing the Alan AI API key within the Alan.js component was not being correctly referenced. This resulted in the component attempting to utilize an empty or incorrect API key, which caused the interactions with the Alan AI platform to fail.

Changes Made:

To resolve this issue, the following changes were implemented:

  • Updated Environment Variable Reference: The reference to the environment variable storing the Alan AI API key within the Alan.js component has been corrected. The component now properly retrieves the API key from the environment variable, ensuring its accurate usage during interactions with the Alan AI platform.

Screenshots

Note to reviewers

@netlify
Copy link

netlify bot commented Aug 8, 2023

Deploy Preview for devsintech ready!

Name Link
🔨 Latest commit 457aafc
🔍 Latest deploy log https://app.netlify.com/sites/devsintech/deploys/64d23c8c3903760008e0bab8
😎 Deploy Preview https://deploy-preview-447--devsintech.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@vercel
Copy link

vercel bot commented Aug 8, 2023

@Nishitbaria is attempting to deploy a commit to the DevsInTech Team on Vercel.

A member of the Team first needs to authorize it.

@Nishitbaria
Copy link
Contributor Author

@Susmita-Dey ,please review and then merge my pull request

@Nishitbaria
Copy link
Contributor Author

Nishitbaria commented Aug 9, 2023

@Susmita-Dey ,please review and then merge my pull request,tell me any other i will resolve

2 similar comments
@Nishitbaria
Copy link
Contributor Author

@Susmita-Dey ,please review and then merge my pull request,tell me any other i will resolve

@Nishitbaria
Copy link
Contributor Author

@Susmita-Dey ,please review and then merge my pull request,tell me any other i will resolve

Copy link
Member

@Susmita-Dey Susmita-Dey left a comment

Choose a reason for hiding this comment

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

LGTM

@Susmita-Dey Susmita-Dey added gssoc23 This label is assigned to those who want to work on an issue under GSSoC23 level1 10 Points (Docs/Minor bugs) labels Aug 10, 2023
@Susmita-Dey Susmita-Dey merged commit a78a2cf into devs-in-tech:main Aug 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gssoc23 This label is assigned to those who want to work on an issue under GSSoC23 level1 10 Points (Docs/Minor bugs)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants