Workspace Chat Details Page Notification Preferences Picker is Broken

[Santhosh-Sellavel]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Action Performed:

1. Go to New Expensify
2. Click +
3. New workspace
4. Invite some users to the workspace
5. Now go to the report list (Chat list)
6. Press CMD + K or click on the search icon
7. Enter announce if workspace channel or room doesn't appear.
8. Open announce chat from search results
9. Click on the chat header
10. Details page will open
11. Look at notification preferences picker its broken

Expected Result:

I don't know exactly. IMO Like this short picker label and description below.

Actual Result:

Workaround:

Can the user still use Expensify without this being fixed? Technically Yes, But it's should be fixed as soon as possible because the component looks broken

Platform:

Where is this issue occurring? ALL Platforms

• Web
• iOS
• Android
• Desktop App
• Mobile Web

Suggestion or Proposal

Need's input from design team how it's intended look, cc: @shawnborton.
IMO is it should look similar to as it is inpreferences page(Attached Image in Expected Result)

Proposal post triage & discussion.

Version Number:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:

View all open jobs on GitHub

[MelvinBot]

Triggered auto assignment to @sophiepintoraetz (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

[shawnborton]

For this one, I think we should do something like this where we have a simple label + value, then hint text below the UI control that explains what the selection does:

[Santhosh-Sellavel]

@shawnborton I think we are okay with this UI, spacing between Picker and hint text is mt1 that is 4

Kindly help me with the following, currently we have the following options in,

If we want to change the options list here, let me know new options name & translations copy for new options relevant value hint text copy.
If we keep the same options as option then relevant value hint text copy with translations alone. Thanks!

[shawnborton]

The spacing looks great. But actually maybe we don't need the hint text, maybe we can just shorten the label to be something like "Notify me about new messages"?

[Santhosh-Sellavel]

So I think we are fine with existing Options, Will update the hint text to "Notify me about new messages", help me with es translations alone. Let's get it done, thanks!

cc: @shawnborton

[sophiepintoraetz]

All looking good - great detail on the write up, Santhosh.

[MelvinBot]

Triggered auto assignment to @nkuoch (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[Santhosh-Sellavel]

@sophiepintoraetz why it's not been external?

[Santhosh-Sellavel]

Is it because just a copy change?

[MelvinBot]

Triggered auto assignment to @trjExpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

[sophiepintoraetz]

Hey @Santhosh-Sellavel - looks like this may have been missed on our end but I've applied the label now, so @trjExpensify will suss out payment for the reporting of the issue. Thanks!

[botify]

🚀 Deployed to staging by @nkuoch in version: 1.0.88-3 🚀

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

[trjExpensify]

Just coming back from OoO. @Santhosh-Sellavel I've created the Upwork job here. Can you apply please, and then I'll pay it and close this issue out.

[Santhosh-Sellavel]

@trjExpensify Applied!

[trjExpensify]

Perfect. Offer sent 👍

[Santhosh-Sellavel]

@trjExpensify Accepted, thanks! 🥳

[trjExpensify]

Awesome, paid. Going to close this out now, thanks @Santhosh-Sellavel!

[botify]

🚀 Deployed to production by @roryabraham in version: 1.0.90-2 🚀

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