[BUG] The product registration does not list all child categories

[leandrosantoss]

Describe the bug
The product registration does not list all child categories

Screenshots

This child category was not found in the category tree in product registration

Clicking to expand "Tabacaria"

In the Data Base the category "Tabacaria" has 75 rows

Background (please complete the following information):

• NodeJS 20
• EverShop 1.2.1

[leandrosantoss]

Hi @treoden I believe it is the same problem that happens in GraphQL. The query is limited and I need to send the "limit" in the filter for it to work.

[leandrosantoss]

@treoden this is a Critical bug for me

Some change was made and it is limiting the system. Here it only lists 20, and in the database I have 114.

[leandrosantoss]

@treoden news about this? This is a critical bug

[omarsoufiane]

I can help by putting a bounty for solving this issue.

[bountyhub-bot]

🚀 Bounty Alert!

💰 A bounty of $30.00 has been created by omarsoufiane on BountyHub for this issue.

🔗 Claim this bounty by submitting a pull request that solves this issue!

Good luck, and happy coding! 💻

[rahat2134]

@leandrosantoss
I understand the bug where only 20 child categories are being displayed in the product registration instead of all 75 categories from the database. This appears to be a pagination/limit issue in the GraphQL query. The solution would involve modifying the GraphQL resolver to either remove the default pagination limit or implement proper pagination for the category tree.
I'd like to work on this issue. Could you please assign it to me?

[leandrosantoss]

This pull request does not solve this problem :(

#654

[risjai]

@leandrosantoss check this out for the fix : #655
I have attached the video for the working solution. Tested it on my system, works well.

screen-recording-2024-10-30-at-30535-pm_lAevZzT9.mp4

Couple of things :

• The issue exists not on just the child categories but direct product categories as well.
• @rahat2134 solution also works but it has many side effects. Eg : Wherever pagination is used like in other pages, it will affect those limits.

Ping me on discord if you have any doubts.

[risjai]

@omarsoufiane @leandrosantoss can we conclude this?

[omarsoufiane]

@risjai the issue is about child categories, not main categories, so your pr doesn't fix it ( judging from the code because I didn't test it yet)

[leandrosantoss]

Can I test in my environment after generating a new release

[risjai]

This fixes the child categories problem as well. Now I feel, I should have made a video directly of child categories but anyways, will wait for Leonardo to test it himself. Leonardo, did you know the problem exists today with parent categories as well?

…

On Tue, 5 Nov 2024 at 11:50 PM, Leandro Pereira dos Santos < ***@***.***> wrote: Can I test in my environment after generating a new release — Reply to this email directly, view it on GitHub <#630 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AIFQEPZ2ZWQB3KJUEUQMA7LZ7ED7VAVCNFSM6AAAAABOMGA7N2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJXHA3TENBYGA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[risjai]

@leandrosantoss Did you test this out?

[risjai]

@omarsoufiane how do we conclude this?

[omarsoufiane]

@risjai the PR isn't merged, and I will test it once it's merged

[treoden]

Hi @leandrosantoss ,

Would you mind trying the last PR?

Thanks

[IIITManjeet]

Hey, I'd like to attempt the bounty.

[treoden]

Completed in v1.2.2