Gatsby-theme-novela: Not able to use shadowing on data.query.js file

[nnnkit]

I am trying to shadow the data.query.js file to get the desired output. But it's not working. Can you please help @ChristopherBiscardi

[brotzky]

@nnnkit could you link me to the repo you're working in? It's possible you have an incorrect path that's causing shadowing to fail.

[wilmanbarrios]

Hi @brotzky, I have the same problem, can you check my src folder to see if I am doing right the shadowing of the data.query.js file?

[brotzky]

That looks like the correct shadowing. This might actually be a bug within Gatsby Theme's Component Shadowing itself.

I am assuming you're exporting correcting from data.query.js?

Correct path:
http://d.ntve.co/JI4fM0

[wilmanbarrios]

Sorry for the late response.

Yes @brotzky, i just copy the data.query.js file from the source, and just did my modification, and then run yarn clean && yarn dev.

But I don't get it to work, do you have any idea way?

Here is the export part.

[wilmanbarrios]

@brotzky I was reading about shadowing, and it sames that we only can shadow files in the src folder, and since the data.query.js file is outside of the src its no going to work.

Is there another way to modify the data.query.js file or a way to change the format and locale of the published date of the articles?

That's my end goal, change the format and locale of the published date of the articles.

[brotzky]

I can move it into the src file if that's an easy win.

I think that's a better approach then somehow making it configurable without shadowing.

[wilmanbarrios]

I think that's a better approach then somehow making it configurable without shadowing.

I think the same, that would be a win.

[wilmanbarrios]

I can move it into the src file if that's an easy win.

Do you need any help @brotzky?

[brotzky]

If you'd like to PR it that would be awesome :)

[wilmanbarrios]

Awesome i'm on it. :+1:

[wilmanbarrios]

Hey @brotzky I just send the PR #176 take a look.

[jmigdelacruz]

Hi @wilmanbarrios has the merged PR fixed this?

I'm trying to do the same as you and update the date format. When i try and shadow the data.query.js file in the same folder you have it on as on your screenshot, still nothing happens even after i do a gatsby build/develop

Any ideas?

[wilmanbarrios]

@jmigdelacruz no dude.

I ended doing a shadowing of the component that render the published date and then applied my custom format based on the the format of the date that is coming from GraphQL.

I think that is something with the path resolution in the theme itself, but i'm not sure.

[brotzky]

Hmmm, im not entirely sure as well. Shadowing is a bit of a blackbox when it doesn't work properly so it's hard for me to fix it.

[LekoArts]

This will be fixed by https://github.com/gatsbyjs/gatsby/pull/17681

[wilmanbarrios]

Looking forward to that PR, thanks.

[wilmanbarrios]

The PR just got merged. But still changing the graphQL query not work.

[brotzky]

Maybe we also have to update Novela's Gatsby version?

[wilmanbarrios]

I'm not sure, but we could try and see it works.

[aabosh]

Any update on this?

[luisarbezerra]

Any update?

[brotzky]

@aabosh and @luisarbezerra you're both welcome to fix it yourselves and submit a PR ❤️