Removing the read_orders scope from list of default scopes provided to a generated app

[TerencedzLi]

With reference to: Issue

[vfonic]

There are couple of more read_orders in the scope throughout the codebase. To find them all, it's the best to do a project-wide search for read_orders.

[TerencedzLi]

AFAIK, the remaining references are either to showcase usage of scopes (in docs) or a part of some tests that allow you to specify scopes to be used (i.e for CLI)

[tylerball]

I don't have a lot of context here but I think we should have a plan to update those docs as well.

[TerencedzLi]

Note to self: Here's one part of the documentation that needs to be updated. https://help.shopify.com/en/api/getting-started/authentication/public-authentication

Node example docs only use read_products. https://help.shopify.com/en/api/tutorials/building-node-app

[jamiemtdwyer]

Note to self: Here's one part of the documentation that needs to be updated. https://help.shopify.com/en/api/getting-started/authentication/public-authentication

Node example docs only use read_products. https://help.shopify.com/en/api/tutorials/building-node-app

I don't see any reason why any documentation on the help site would need to be updated, including either of these.

There are no tutorials or guides on the help site that use shopify_app.

[TerencedzLi]

Note to self: Here's one part of the documentation that needs to be updated. https://help.shopify.com/en/api/getting-started/authentication/public-authentication
Node example docs only use read_products. https://help.shopify.com/en/api/tutorials/building-> node-app

I don't see any reason why any documentation on the help site would need to be updated, including either of these.

There are no tutorials or guides on the help site that use shopify_app.

Yeaup, those links I put are irrelevant. But as a whole effort, as seen in the initial issue, we wanted to remove read_orders as a default scope. I agree probably not relevant to this gem :) thanks for the feedback