feat(Message): replace reply functionality with inline replies #4870
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please describe the changes this PR makes and why it should be merged:
original:
This PR removes Message#reply from the library as it is just a convenience shortcut for
Before it was argued that the added benefit of adding the reply recipient as parsed mention to the respective options was indeed a considerable ease of use compared to the #send call with all required options to make this work, however Discord has recently had previews for inline replies in development client and rumors of a new message type (19) being the proper implementation after the initial tests have reached us.
While this is not yet officially confirmed in any discord API documentation cleaning the namespace to provide actual reply functionality (should it be available for bots in the first place) or keeping it clear due to the associated meaning of the term "reply" brought forward by aforementioned functionality.
The implementation of the feature itself can then take place in a minor release, while the clearing of the name space (this PR) obviously requires a semver: major, thus me making this PR now, aiming it for v13 together with using the new API versions.
Should I have missed any caveat please do let me know!
Status
Semantic versioning classification: