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

Introduce room v4 which updates event ID format. #5217

Merged
merged 1 commit into from
May 21, 2019
Merged

Conversation

richvdh
Copy link
Member

@richvdh richvdh commented May 21, 2019

Will keep this unmerged until matrix-org/matrix-spec-proposals#2002 completes FCP.

Copy link
Member

@erikjohnston erikjohnston left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, though will this cause my server to explode because I'm in a room it no longer knows about?

@codecov
Copy link

codecov bot commented May 21, 2019

Codecov Report

Merging #5217 into develop will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           develop    #5217   +/-   ##
========================================
  Coverage    62.48%   62.48%           
========================================
  Files          338      338           
  Lines        35035    35035           
  Branches      5734     5734           
========================================
  Hits         21890    21890           
- Misses       11596    11598    +2     
+ Partials      1549     1547    -2

@richvdh
Copy link
Member Author

richvdh commented May 21, 2019

Sure, though will this cause my server to explode because I'm in a room it no longer knows about?

I dunno. What's the alternative? that we keep test versions for ever?

@richvdh
Copy link
Member Author

richvdh commented May 21, 2019

[I'm kinda hoping that it will fail somewhat gracefully. I guess i should check...]

@richvdh
Copy link
Member Author

richvdh commented May 21, 2019

#5219 makes the fail better.

@richvdh richvdh merged commit 17f6804 into develop May 21, 2019
@richvdh richvdh deleted the rav/room_v4 branch May 21, 2019 15:23
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