Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Drop support for v2_alpha API prefix #5190

Merged
merged 2 commits into from
May 15, 2019
Merged

Drop support for v2_alpha API prefix #5190

merged 2 commits into from
May 15, 2019

Conversation

richvdh
Copy link
Member

@richvdh richvdh commented May 15, 2019

No description provided.

@richvdh richvdh requested a review from a team May 15, 2019 09:31
@codecov
Copy link

codecov bot commented May 15, 2019

Codecov Report

Merging #5190 into develop will decrease coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #5190      +/-   ##
===========================================
- Coverage    62.15%   62.14%   -0.01%     
===========================================
  Files          336      336              
  Lines        34659    34656       -3     
  Branches      5694     5693       -1     
===========================================
- Hits         21541    21538       -3     
  Misses       11583    11583              
  Partials      1535     1535

@erikjohnston
Copy link
Member

Out of interest have we checked how often (or not) we see /v2_alpha requests on matrix.org?

@richvdh
Copy link
Member Author

richvdh commented May 15, 2019

Yeah there are quite a few /sync requests coming from unidentified Android UAs. They certainly aren't from riot-android. My feeling is that at this point any app using those APIs ought to have been fixed long ago, so I think it's time to move on.

@richvdh richvdh merged commit 5f027a3 into develop May 15, 2019
@richvdh richvdh deleted the rav/kill_v2_alpha branch December 1, 2020 12:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants