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.

Sign up for GitHub

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 from summary url if no ai env var #9895

Merged

nickoferrall merged 1 commit into master from fix/remove-ai-url-param

Jul 1, 2024

Contributor

nickoferrall commented Jun 27, 2024

Discovered by @Dschoordsch here

To test

Remove OpenAI env var
End a retro meeting
See that the summary does not have ai=true in the URL


fix: remove ai from summary url if no ai env var

281c864

github-actions bot added the size/xs label

nickoferrall requested a review from Dschoordsch

June 27, 2024 16:56

Dschoordsch approved these changes

View reviewed changes

nickoferrall merged commit 4413142 into master

6 checks passed

nickoferrall deleted the fix/remove-ai-url-param branch

July 1, 2024 09:11

parabol-release-bot bot mentioned this pull request

chore(release): release v7.38.0 #9905

Merged

github-actions bot mentioned this pull request

chore(release): Test v7.38.0 #9921

Merged

24 tasks

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