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: Pull fileNode file name from airtable metadata #374

Merged
merged 6 commits into from
May 21, 2022

Conversation

ryan-talus
Copy link
Contributor

@ryan-talus ryan-talus commented Dec 9, 2021

This makes the fileNode pull name from the airtable
metadata instead of the remote file, because when an airtable
user changes the file name, airtable does not rename the original
file. This change makes file name changes in airtable usable in
the fileNode instead of needing to download, rename, and
re-upload the file.

This makes the fileNode pull its name from the airtable
metadata instead of the remote file, because when an airtable
user changes the file name, airtable does not rename the original
file. This change makes file name changes in airtable usable in
the fileNode instead of needing to download, rename, and
re-upload the file.
@ryan-talus ryan-talus changed the title Pull fileNode file name from airtable metadata Fix: Pull fileNode file name from airtable metadata Dec 10, 2021
@jbolda
Copy link
Owner

jbolda commented Dec 13, 2021

Thanks for this! Do you know if this option was added at a later date? Considering which versions of Gatsby we are able to support with this change.

I think regardless we can figure out how to best land this, so would you also add a change file? 🙏

@ryan-talus
Copy link
Contributor Author

@jbolda Thanks!

I've added the change file.

The name option was added in this PR:
gatsbyjs/gatsby#11054

Which was merged Jan 23, 2019, into [email protected], So anything past gatsby v2.26 should work?

Let me know if there's anything else I should add to this PR!

@ryan-talus
Copy link
Contributor Author

@jbolda Is there anything else helpful I could add to this pull request?

@ryan-talus
Copy link
Contributor Author

@jbolda Is there anything I could do to help push this forward?

@jbolda
Copy link
Owner

jbolda commented Apr 15, 2022

Sorry, all my attention has been on driving other projects forward in a big way.

Yea, with those requirements, this should be fine as a patch. Let's get this merged in. Thanks again!

@all-contributors please add @ryan-talus for code

bors merge

@allcontributors
Copy link
Contributor

@jbolda

I've put up a pull request to add @ryan-talus! 🎉

bors bot added a commit that referenced this pull request Apr 15, 2022
374: Fix: Pull fileNode file name from airtable metadata r=jbolda a=ryan-talus

This makes the fileNode pull `name` from the airtable
metadata instead of the remote file, because when an airtable
user changes the file name, airtable does not rename the original
file. This change makes file name changes in airtable usable in
the fileNode instead of needing to download, rename, and
re-upload the file.

Co-authored-by: Ryan <[email protected]>
Co-authored-by: Ryan Zimmerman <[email protected]>
@bors
Copy link
Contributor

bors bot commented Apr 15, 2022

Build failed:

bors bot added a commit that referenced this pull request Apr 15, 2022
416: docs: add ryan-talus as a contributor for code r=jbolda a=allcontributors[bot]

Add `@ryan-talus` as a contributor for code.

This was requested by jbolda [in this comment](#374 (comment))

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Copy link
Owner

@jbolda jbolda left a comment

Choose a reason for hiding this comment

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

Sorry it took so long to get back to this. I took some vacation this week, so finally have some time to get back to resolving this. When we went to merge it in, the CI failed on Windows. I think with these suggestions, we can close this out.

gatsby-node.js Outdated Show resolved Hide resolved
gatsby-node.js Outdated Show resolved Hide resolved
.changes/fix-airtable-filenames.md Outdated Show resolved Hide resolved
@jbolda
Copy link
Owner

jbolda commented May 21, 2022

I went ahead and applied the suggestions. Doesn't feel like it is worth dragging it out as I have made y'all wait long enough.

bors merge

@bors
Copy link
Contributor

bors bot commented May 21, 2022

Build succeeded:

@bors bors bot merged commit 303fe17 into jbolda:main May 21, 2022
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.

2 participants