Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

runfix: problem with existing database entries #16020

Closed

Conversation

aweiss-dev
Copy link
Member

Reverts #16017

@codecov
Copy link

codecov bot commented Oct 17, 2023

Codecov Report

Merging #16020 (b31808d) into dev (2a7480c) will increase coverage by 0.00%.
Report is 1 commits behind head on dev.
The diff coverage is 50.00%.

@@           Coverage Diff           @@
##              dev   #16020   +/-   ##
=======================================
  Coverage   44.78%   44.79%           
=======================================
  Files         701      701           
  Lines       23300    23302    +2     
  Branches     5291     5291           
=======================================
+ Hits        10435    10437    +2     
- Misses      11504    11505    +1     
+ Partials     1361     1360    -1     

@atomrc atomrc changed the title fix: problem with existing database entries runfix: problem with existing database entries Oct 17, 2023
@aweiss-dev aweiss-dev enabled auto-merge (squash) October 17, 2023 12:49
@@ -370,7 +369,7 @@ describe('ConversationMapper', () => {
status: 0,
team_id: teamId,
type: 0,
verification_state: ConversationVerificationState.UNVERIFIED,
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe this was a nice change, why removing it? 🤔

@@ -61,7 +60,7 @@ export interface SelfStatusUpdateDatabaseData {
otr_muted_status: number;
receipt_mode: number;
status: number;
verification_state: ConversationVerificationState;
verification_state: number;
Copy link
Contributor

Choose a reason for hiding this comment

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

This was also a valid change

@aweiss-dev aweiss-dev disabled auto-merge October 17, 2023 12:52
@aweiss-dev aweiss-dev closed this Oct 17, 2023
@aweiss-dev aweiss-dev deleted the revert-16017-runfix/missing_degredation_warning branch October 17, 2023 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants