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

✨ New Feature - Incremental Static Regeneration #1098

Closed
wants to merge 23 commits into from

Conversation

Harry-Ross
Copy link
Contributor

@Harry-Ross Harry-Ross commented Jul 21, 2023

  • Changed the getting of livestream event information from an API route to getStaticProps w/ ISR

Fixes #1076

Affected routes: All

@Harry-Ross Harry-Ross changed the title Refactored to have livestream info come from getstaticprops ✨ New Feature - Incremental Static Regeneration Jul 25, 2023
@Harry-Ross
Copy link
Contributor Author

image

This logging line means that the SharePoint environment variables cannot be accessed in getStaticProps

@github-actions
Copy link
Contributor

Deployed changes to https://app-sswwebsite-9eb3-pr-1098.azurewebsites.net

@github-actions
Copy link
Contributor

PageSpeed Insights

Category Score
Accessibility 89
Best practices 92
Performance 33
SEO 100

View the full report https://pagespeed.web.dev/report?url=https://app-sswwebsite-9eb3-pr-1098.azurewebsites.net

@Harry-Ross Harry-Ross marked this pull request as ready for review July 28, 2023 02:03
Comment on lines +92 to +107
$GoogleRecaptchaSiteKey = (az keyvault secret show --name Google-Recaptcha-Site-KEY --vault-name ${{ env.KEY_VAULT }} --query value -o tsv)
$MICROSOFT_OAUTH_TENANT_ID = (az keyvault secret show --name MICROSOFT-OAUTH-TENANT-ID --vault-name ${{ env.KEY_VAULT }} --query value -o tsv)
$MICROSOFT_OAUTH_CLIENT_ID = (az keyvault secret show --name MICROSOFT-OAUTH-CLIENT-ID --vault-name ${{ env.KEY_VAULT }} --query value -o tsv)
$MICROSOFT_OAUTH_CLIENT_SECRET = (az keyvault secret show --name MICROSOFT-OAUTH-CLIENT-SECRET --vault-name ${{ env.KEY_VAULT }} --query value -o tsv)
$SHAREPOINT_SITE_ID = (az keyvault secret show --name SHAREPOINT-SITE-ID --vault-name ${{ env.KEY_VAULT }} --query value -o tsv)
$SHAREPOINT_EVENTS_LIST_ID = (az keyvault secret show --name SHAREPOINT-EVENTS-LIST-ID --vault-name ${{ env.KEY_VAULT }} --query value -o tsv)
$SHAREPOINT_EXTERNAL_PRESENTERS_LIST_ID = (az keyvault secret show --name SHAREPOINT-EXTERNAL-PRESENTERS-LIST-ID --vault-name ${{ env.KEY_VAULT }} --query value -o tsv)

echo "GoogleRecaptchaSiteKey=$GoogleRecaptchaSiteKey" >> $env:GITHUB_OUTPUT
Write-Host '✅ KV - Secret retrieved'
echo "MICROSOFT_OAUTH_TENANT_ID=$MICROSOFT_OAUTH_TENANT_ID" >> $env:GITHUB_OUTPUT
echo "MICROSOFT_OAUTH_CLIENT_ID=$MICROSOFT_OAUTH_CLIENT_ID" >> $env:GITHUB_OUTPUT
echo "MICROSOFT_OAUTH_CLIENT_SECRET=$MICROSOFT_OAUTH_CLIENT_SECRET" >> $env:GITHUB_OUTPUT
echo "SHAREPOINT_SITE_ID=$SHAREPOINT_SITE_ID" >> $env:GITHUB_OUTPUT
echo "SHAREPOINT_EVENTS_LIST_ID=$SHAREPOINT_EVENTS_LIST_ID" >> $env:GITHUB_OUTPUT
echo "SHAREPOINT_EXTERNAL_PRESENTERS_LIST_ID=$SHAREPOINT_EXTERNAL_PRESENTERS_LIST_ID" >> $env:GITHUB_OUTPUT

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Try to condense KV block

@Harry-Ross
Copy link
Contributor Author

As per my conversation with @wicksipedia, there may be an alternate soltuion with next/dynamic that does not involve the additional complexity that ISR adds.

@Harry-Ross Harry-Ross closed this Jul 28, 2023
@github-actions
Copy link
Contributor

Deployed changes to https://app-sswwebsite-9eb3-pr-1098.azurewebsites.net

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

⏩ Performance - LiveStream Banner Worsening Performance
2 participants