ricecooker icon indicating copy to clipboard operation
ricecooker copied to clipboard

Add many-children warning and validation checks

Open ivanistheone opened this issue 5 years ago • 2 comments

Description

Topic nodes that contain hundreds of child nodes are difficult to load (lots of json data) and render (lots of dom elements)

For example, this topic takes 2min+ to load on a good internet connection and powerful browser: http://kolibridemo-ar.learningequality.org/en/learn/#/topics/t/987577eab2c35f8a855d5b3bf6f8d44c

Proposed solution

Add validation logic when creating the channel that prevents topics with many children to be created:

  • if len(children) > 50: warning

  • if len(children) > 200: error, stop chef run

values 50 and 200 to be discussed

ivanistheone avatar Mar 11 '20 15:03 ivanistheone

Hi @ivanistheone,

I'm interested in working on this issue. As someone relatively new to contributing, I’ll do my best to implement the validation checks and test them to improve performance. Let me know if there are any specific guidelines to follow.

Yadnyesh-More avatar Nov 10 '24 06:11 Yadnyesh-More

Hi @Yadnyesh-More, thanks for volunteering. This issue needs decisions and is not ready for contribution.

There are contributing opportunities in three repositories. See the contributing guidelines including links to issues suitable for contribution for each repository here:

You can also see the list of unassigned "help wanted" or "good first issue" issues across all repositories.

MisRob avatar Nov 11 '24 09:11 MisRob