Update custom js content

[mrtuvn]

Purpose of this pull request

This pull request (PR) update content for this topic
This content is make misunderstand for use. We should update it to avoid confusing for reader/devs

Recall this issue magento/magento2#24830

Affected DevDocs pages

https://devdocs.magento.com/guides/v2.3/javascript-dev-guide/javascript/custom_js.html

Related docs

https://devdocs.magento.com/guides/v2.3/javascript-dev-guide/javascript/requirejs.html#requirejs-config-deps

Links to Magento source code

• ...

[devops-devdocs]

An admin must run tests on this PR before it can be merged.

[rogyar]

I'm not sure what is this section about but the deps part of requirejs-config is being merged across all modules/themes. So, technically, we cannot override something in that way. Removing it might be a good idea but maybe there's something behind the scenes that the original author wanted to say. Still not sure.

[mrtuvn]

@dshevtsov @rafaelstz Can you guys give me some comments
@rogyar yes i know that but i don't think we can do much with js only when all requirejs-config merged at one. For disable js we need more help with php code. But this seem out of scope of this topic

[rogyar]

Still not able to guess the original intention of this description. Well then, considering the fact that this approach is technically is not correct, I assume that it's better to remove this information to avoid further confusion.

[mrtuvn]

@guz-anton What do you think about this update

[ghost]

Hi @mrtuvn, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.