Conditionally hide possibility to change password for SIWE accounts
ClosedPublic
Actions

Authored by marcin on Dec 29 2022, 8:11 AM.

Details

Reviewers

tomek
atul

Commits

rCOMM170e8fede586: Conditionally hide possibility to change password for SIWE accounts

Summary

This differential disables UI that allows to change password for SIWE accounts.

Test Plan

Create SIWE accounts. Button to enter passowrd edition panel should simply not be visible, but everything should remain as it was for conventional accounts.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

marcin created this revision.Dec 29 2022, 8:11 AM

Herald added a subscriber: ashoat. · View Herald TranscriptDec 29 2022, 8:11 AM

marcin published this revision for review.Dec 29 2022, 8:12 AM

marcin added a parent revision: D6103: Do not prompt for password when deleting thread.

Harbormaster failed remote builds in B14928: Diff 20345!Dec 29 2022, 8:15 AM

marcin added a child revision: D6105: Do not prompt for password when deleting thread on web.Dec 29 2022, 8:42 AM

Rebase master

Harbormaster completed remote builds in B14939: Diff 20357.Dec 29 2022, 9:02 AM

Rebase

Harbormaster completed remote builds in B14971: Diff 20422.Dec 30 2022, 3:51 AM

tomek accepted this revision.Dec 30 2022, 3:56 AM

This revision is now accepted and ready to land.Dec 30 2022, 3:56 AM

Closed by commit rCOMM170e8fede586: Conditionally hide possibility to change password for SIWE accounts (authored by marcin). · Explain WhyDec 30 2022, 4:13 AM

This revision was automatically updated to reflect the committed changes.

marcin added a commit: rCOMM170e8fede586: Conditionally hide possibility to change password for SIWE accounts.

ashoat added inline comments.Dec 30 2022, 6:32 AM

native/profile/profile-screen.react.js
104	This should've been `passwordEditingUI`, "edition" doesn't mean what you think

marcin added inline comments.Dec 30 2022, 7:13 AM

native/profile/profile-screen.react.js
104	Yes - language mistake from my side. I should create a small diff that fixes it shortly.

Revision Contents
Changeset List

Path

Size

native/

profile/

profile-screen.react.js

36 lines

Diff 20431

View Options

native/profile/profile-screen.react.js

Conditionally hide possibility to change password for SIWE accountsClosedPublicActions