edge_ngram is now edgeNGram?

[ifo20]

I had errors trying to PUT mapping with the syntax as per this documentation. From stackoverflow answers I saw different spelling and treatment of min/max as strings rather than ints. Is this update required?

• Have you signed the contributor license agreement?
• Have you followed the contributor guidelines?
• If submitting code, have you built your formula locally prior to submission with gradle check?
• If submitting code, is your pull request against master? Unless there is a good reason otherwise, we prefer pull requests against master and will backport as needed.
• If submitting code, have you checked that your submission is for an OS that we support?
• If you are submitting this code for a class then read our policy for that.

[jimczi]

Both form ( edge_ngram and edgeNGram) should be accepted but we want to deprecate the camel case form because it is not consistent with the naming of the other filters:
#30209
I hope you don't mind if I close your pr, the documentation is correct and we have tests that check the execution of these snippets. If you have errors with this syntax I suggest that you open an issue first with a complete recreation.

[ifo20]

Ah my mistake. Thanks for the explanation - indeed I did manage to get it working with edge_ngram in the end so must have been some other bug I had