Skip to content

feat(maevsi): add openai api key to environment variables#162

Merged
dargmuesli merged 3 commits intomasterfrom
feat/env/api-keys
Feb 7, 2025
Merged

feat(maevsi): add openai api key to environment variables#162
dargmuesli merged 3 commits intomasterfrom
feat/env/api-keys

Conversation

@HMDank
Copy link
Copy Markdown
Contributor

@HMDank HMDank commented Feb 6, 2025

I added outr OpenAI's API keys as an env variable

@HMDank HMDank requested a review from dargmuesli February 6, 2025 23:12
@dargmuesli dargmuesli changed the title feat(keys): add openai api to env feat(maevsi): add openai api key to environment variables Feb 6, 2025
Copy link
Copy Markdown
Member

@dargmuesli dargmuesli left a comment

Choose a reason for hiding this comment

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

Looking good! Let's just keep the alphabetical sorting everywhere (the new key goes after MAEVSI_NUXT_PRIVATE_API_NOTIFICATION_SECRET).

@HMDank HMDank requested a review from dargmuesli February 6, 2025 23:26
@HMDank HMDank requested a review from dargmuesli February 7, 2025 00:00
Copy link
Copy Markdown
Member

@dargmuesli dargmuesli left a comment

Choose a reason for hiding this comment

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

Great!

@dargmuesli dargmuesli merged commit c511f0f into master Feb 7, 2025
5 checks passed
@dargmuesli dargmuesli deleted the feat/env/api-keys branch February 7, 2025 00:14
maevsi-bot pushed a commit that referenced this pull request Feb 7, 2025
## [8.4.0](8.3.8...8.4.0) (2025-02-07)

### Features

* **maevsi:** add openai api key to environment variables ([#162](#162)) ([c511f0f](c511f0f))
@maevsi-bot
Copy link
Copy Markdown

🎉 This PR is included in version 8.4.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants