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

[FIX] Anonymous user buttons #27551

Merged
merged 19 commits into from
Jan 10, 2023
Merged

[FIX] Anonymous user buttons #27551

merged 19 commits into from
Jan 10, 2023

Conversation

hugocostadev
Copy link
Contributor

@hugocostadev hugocostadev commented Dec 15, 2022

Proposed changes (including videos or screenshots)

When the server accepts an anonymous user to join and chat without login, two buttons appears: one for Login and the other to send messages without login.

The component was rewritten to use the fuselage Design System and native hooks to perform actions.

image

Issue(s)

closes #27149

fixes https://forums.rocket.chat/t/sign-in-to-start-talking-and-or-talk-as-anonymous-buttons-dont-work/15358

Steps to test or reproduce

Further comments

TC-318

@hugocostadev hugocostadev requested a review from a team as a code owner December 15, 2022 12:09
@codecov
Copy link

codecov bot commented Dec 15, 2022

Codecov Report

Merging #27551 (04daa6b) into develop (b2b6fbb) will increase coverage by 1.56%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #27551      +/-   ##
===========================================
+ Coverage    40.92%   42.48%   +1.56%     
===========================================
  Files          837      811      -26     
  Lines        17647    17133     -514     
  Branches      1984     1909      -75     
===========================================
+ Hits          7222     7279      +57     
+ Misses       10186     9611     -575     
- Partials       239      243       +4     
Flag Coverage Δ
e2e 42.48% <100.00%> (+1.56%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Copy link
Member

@gabriellsh gabriellsh left a comment

Choose a reason for hiding this comment

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

Nice work, this fixes it, but can you add E2E tests for it? Seems like an easy win.

@hugocostadev hugocostadev requested a review from a team as a code owner December 26, 2022 16:05
gabriellsh
gabriellsh previously approved these changes Jan 3, 2023
@dionisio-bot dionisio-bot bot added the stat: ready to merge PR tested and approved waiting for merge label Jan 5, 2023
@hugocostadev hugocostadev added this to the 6.0.0 milestone Jan 5, 2023
@hugocostadev hugocostadev requested review from tassoevan and removed request for tassoevan January 5, 2023 18:34
@kodiakhq kodiakhq bot merged commit 605c762 into develop Jan 10, 2023
@kodiakhq kodiakhq bot deleted the fix/anonymous-user branch January 10, 2023 04:29
gabriellsh added a commit that referenced this pull request Jan 11, 2023
…customRolesUpsell

* 'develop' of github.com:RocketChat/Rocket.Chat:
  Regression: Composer Toolbox overlapping borders (#27714)
  [FIX] Add the possibility of danger style prop to close button on UiKit contextual-bar component (#27533)
  [FIX] Discussion visibility when inside a private team. (#27601)
  Chore: Restrict legacy message events (#27691)
  [FIX] Anonymous user buttons (#27551)
  [BREAK] [FIX] VideoConference Ringer working intermittently on microservices (#27703)
  [BREAK] Update Meteor to 2.9 and NodeJS to 14.21.1 (#27639)
@sampaiodiego sampaiodiego mentioned this pull request Feb 17, 2023
@sampaiodiego sampaiodiego mentioned this pull request Mar 9, 2023
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
squad: team-collab stat: ready to merge PR tested and approved waiting for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

anonymous message writing does not work
6 participants