Skip to content

Revert "Avatar doesn't load when login from public room as existing user"#77031

Merged
lakchote merged 1 commit into
mainfrom
revert-74819-fix/74176
Dec 8, 2025
Merged

Revert "Avatar doesn't load when login from public room as existing user"#77031
lakchote merged 1 commit into
mainfrom
revert-74819-fix/74176

Conversation

@lakchote

@lakchote lakchote commented Dec 8, 2025

Copy link
Copy Markdown
Contributor

@lakchote lakchote requested review from a team as code owners December 8, 2025 18:57
@melvin-bot melvin-bot Bot requested review from heyjennahay and srikarparsi and removed request for a team December 8, 2025 18:57
@melvin-bot

melvin-bot Bot commented Dec 8, 2025

Copy link
Copy Markdown

@srikarparsi Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

@lakchote lakchote merged commit 5bc17e4 into main Dec 8, 2025
31 of 33 checks passed
@lakchote lakchote deleted the revert-74819-fix/74176 branch December 8, 2025 18:59
@melvin-bot melvin-bot Bot added the Emergency label Dec 8, 2025
@melvin-bot

melvin-bot Bot commented Dec 8, 2025

Copy link
Copy Markdown

@lakchote looks like this was merged without a test passing. Please add a note explaining why this was done and remove the Emergency label if this is not an emergency.

@lakchote

lakchote commented Dec 8, 2025

Copy link
Copy Markdown
Contributor Author

Not emergency. Straight revert.

@lakchote lakchote removed the Emergency label Dec 8, 2025
@codecov

codecov Bot commented Dec 8, 2025

Copy link
Copy Markdown

Codecov Report

✅ Changes either increased or maintained existing code coverage, great job!

Files with missing lines Coverage Δ
.../components/CurrentUserPersonalDetailsProvider.tsx 100.00% <100.00%> (ø)
... and 180 files with indirect coverage changes

@OSBotify

OSBotify commented Dec 8, 2025

Copy link
Copy Markdown
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@IuliiaHerets

Copy link
Copy Markdown

@lakchote do we need to QA this?

@OSBotify

OSBotify commented Dec 9, 2025

Copy link
Copy Markdown
Contributor

🚀 Deployed to staging by https://github.com/lakchote in version: 9.2.74-0 🚀

platform result
🖥 desktop 🖥 success ✅
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 cancelled 🔪

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by https://github.com/yuwenmemon in version: 9.2.74-12 🚀

platform result
🖥 desktop 🖥 success ✅
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 failure ❌

1 similar comment
@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by https://github.com/yuwenmemon in version: 9.2.74-12 🚀

platform result
🖥 desktop 🖥 success ✅
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 failure ❌

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by https://github.com/yuwenmemon in version: 9.2.74-12 🚀

platform result
🖥 desktop 🖥 success ✅
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants