fixed introspection field "specifiedBy" to "specifiedByURL"

[Code-Hex]

• I've sent PR to graphql/graphql-js is modified introspection field according to "[RFC] Custom Scalar Specification URLs".
  • But I have got a reply from @IvanGoncharov and I understand specifiedBy should be specifiedByURL to be consistent with deprecatedReason representing @deprecated(reason: "") directive. (I agreed this direction)
  • Use specifiedBy instead of specifiedByUrl graphql-js#3032 (comment)

[IvanGoncharov]

@Code-Hex Thanks for opening it 👍

Context: #649 (comment)
So naming change from the above suggestion was accepted by the introspection field stayed unchanged.
The only argument I saw is:

This is a personal feelings, specifiedByURL causes some associations with the Hungarian notation, which has its drawbacks.

But graphql-js PR added specifiedByURL and I always assumed that #649 is also used that name.
Moreover, other implementations (e.g. Python, graphql-java) followed reference implementation and did specifiedByURL.

Also, I think specifiedByURL is needed to be consistent with deprecatedReason and future directives exposed through introspection.
The original proposal is still Draft - Stage3 so we can fix it before it gets released.

cc: @leebyron

[leebyron]

Thanks for opening this - I thought we had handled this, but perhaps we missed something.

While you're at it, can you make sure prose is updated accordingly?

Example:

When defining a custom scalar, GraphQL services should provide a specification URL via the @specifiedBy directive or the specifiedBy introspection field.

to

When defining a custom scalar, GraphQL services should provide a specification URL via the @specifiedBy directive or the specifiedByURL introspection field.

[Code-Hex]

@leebyron I've done it. Could you review please.

[nodkz]

I think you made a typo with specifiedByURL, should be specifiedByUrl.

GraphQLJSON https://github.com/taion/graphql-type-json/blob/master/src/index.js#L55-L56 and other scalars distributed via NPM now lost specifiedByUrl property with v16.0.0-alpha.2

PS. The opened issue for this typo: graphql/graphql-js#3156