-
Notifications
You must be signed in to change notification settings - Fork 1
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: Add ability for Netlify to hold API key as environment #14
Conversation
✅ Deploy Preview for hutasker ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
This pull request is split into 5 parts for easier review. Changed files are located in these folders:
|
</div> | ||
{{end}} | ||
<script> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Am proud of this script, and will leave it here for now as am using Hugo to inject data into an array
{{$user := $airtableuser.records}} | ||
{{with .Creator}} | ||
{{$creator:=index . 0}} | ||
{{range where $user "id" $creator}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There may be a cleaner way to write this using scratch sets at the top, however it works in this context and am happy for now
Fixed netlify build script. Note that building with netlify env variables need to be outside quotation marks (''). |
No description provided.