-
Notifications
You must be signed in to change notification settings - Fork 984
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
[ISSUE #3313] Migrated public chat creation flow #3381
Conversation
(update :db navigation/replace-view :home)))) | ||
|
||
(handlers/register-handler-fx | ||
:delete-chat? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe :show-delete-chat-confirmation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't use ?
yet but could be a convention for handler name? -confirmation
looks pretty long :)
src/status_im/chat/events.cljs
Outdated
:delete-chat? | ||
[re-frame/trim-v] | ||
(fn [_ [chat-id group?]] | ||
{:show-confirmation {:title (str (i18n/label :t/delete) "?") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is "?" using in all languages?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jeluard ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh my bad, good point. Will make this label too.
src/status_im/chat/handlers.cljs
Outdated
(re-frame/dispatch [:remove-chat current-chat-id])))) | ||
|
||
(handlers/register-handler-fx | ||
:leave-group-chat? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe :show-leave-group-chat-confirmation
?
:clear-history | ||
(fn [{{:keys [current-chat-id] :as db} :db} _] | ||
{:db (assoc-in db [:chats current-chat-id :messages] {}) | ||
:delete-messages current-chat-id})) | ||
|
||
(handlers/register-handler-fx | ||
:clear-history? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
show-clear-history-confirmation
?
Automated test results:test_contact_profile_view:white_check_mark::Test Steps & Error message:
test_browse_link_entering_url_in_dapp_view:white_check_mark::Test Steps & Error message:
test_transaction_send_command_group_chat:x:Test Steps & Error message:
test_group_chat_messages:x:Test Steps & Error message:
test_transaction_send_command_wrong_password:white_check_mark::Test Steps & Error message:
test_send_eth_from_wallet_sign_now:white_check_mark::Test Steps & Error message:
test_send_transaction_from_daap:white_check_mark::Test Steps & Error message:
test_send_eth_from_wallet_sign_later:white_check_mark::Test Steps & Error message:
test_send_eth_to_request_in_group_chat:x:Test Steps & Error message:
test_send_stt_from_wallet_via_enter_contact_code:white_check_mark::Test Steps & Error message:
test_public_chat:white_check_mark::Test Steps & Error message:
test_one_to_one_chat_messages_and_delete_chat:x:Test Steps & Error message:
test_transaction_send_command_one_to_one_chat:x:Test Steps & Error message:
test_send_eth_to_request_in_one_to_one_chat:white_check_mark::Test Steps & Error message:
test_send_eth_to_request_from_wallet:white_check_mark::Test Steps & Error message:
|
d7a8576
to
360917c
Compare
Branch: PR-3381 |
Please also add accessibility lables here cc: @lukaszfryc |
To follow up on @asemiankevich comment -- we want to kick off iOS end-end automation and to do it efficiently we need accessibility labels for elements user may interact with (buttons, list items, menu options, etc). Whenever UI changes we should consider updating the labels. Here is example of accessibility labels we want to introduce in another part of the app #3394 |
875136d
to
2fbc224
Compare
@asemiankevich Great thanks! Can we left the alignment issue out? I will fix it part of #3372 |
@jeluard yes, for sure! |
61aa845
to
eebfed3
Compare
Signed-off-by: Julien Eluard <julien.eluard@gmail.com>
eebfed3
to
33f1d8d
Compare
fixes #3313
fixes #3341
Note that input errors have not been migrated to new paradigm.
Summary:
Migrated public chat creation flow to latest mockups. Also added popup prompt to destructive actions (applies to group chat too)
Steps to test:
status: ready