chore(gatsby): migrate query/redirectWriter to TypeScript #23785
Conversation
gatsbot
bot
added
the
status: triage needed
| const debouncedWriteRedirects = _.debounce(() => { | ||
| // Don't write redirects again until bootstrap has finished. | ||
| if ( | ||
| bootstrapFinished && | ||
| !_.isEqual(oldRedirects, store.getState().redirects) | ||
| ) { | ||
| writeRedirects() | ||
| oldRedirects = store.getState().Redirects | ||
| oldRedirects = store.getState().redirects |
There was a problem hiding this comment.
I corrected this code because I think it's a typo
There was a problem hiding this comment.
That's nice catch!
But there is some asterisk regarding this entire file - we actually don't use it anywhere 😱
It seems after we moved this file to https://github.com/gatsbyjs/gatsby/blob/master/packages/gatsby/src/bootstrap/redirects-writer.js there was some git messup and old file location stayed. What's really weird is that the file we actually use - don't even have that check
|
Hey @taejs! This seems to be duplicate of #23332. I'm just looking, at that was not tracked in typescript migration umbrella issue - so sorry about that. Given that you are already member of gatsby org because of previous pull requests I would like to give a chance to other PR as this is author's first if that's fine with you? |
|
Sure! no problem |
wardpeet
removed
the
status: triage needed
Description
Migrate query/redirectWriter to TypeScript
Related Issues
Related to #21995