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

fix: remove AI UI if user doesn't have access to AI #9856

Merged
merged 2 commits into from
Jun 25, 2024

Conversation

nickoferrall
Copy link
Contributor

Fix #9822

Demo: https://www.loom.com/share/9cc4e5ff56134197ae0e7295359c9dbf

To test

  • Run the app normally and see the AI Icebreaker, discussion question, and AI meeting summary
  • Comment out the OPEN_AI_API_KEY env var in your .env
  • Run yarn predeploy
  • Create a retro meeting and see that the AI icebreaker, discussion question, and AI meeting summary do not show up

@@ -40,19 +40,25 @@ const WholeMeetingSummary = (props: Props) => {
`,
meetingRef
)
const hasAI = window.__ACTION__.hasOpenAI
Copy link
Contributor

Choose a reason for hiding this comment

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

-1 This does not work for the email summary

"time":"2024-06-24T12:59:02.466Z","level":"info","message":"ReferenceError: window is not defined\n    at WholeMeetingSummary (/

@nickoferrall nickoferrall requested a review from Dschoordsch June 24, 2024 15:20
@nickoferrall nickoferrall merged commit 95431b2 into master Jun 25, 2024
5 checks passed
@nickoferrall nickoferrall deleted the fix/9822/hide-ai-ui branch June 25, 2024 15:22
@github-actions github-actions bot mentioned this pull request Jun 25, 2024
24 tasks
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.

Implement ability to hide AI features in self-hosted deployment
2 participants