-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[$250] Update all references from "workspace chat" to "expense chat" #60984
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
Comments
Triggered auto assignment to @maddylewis ( |
ProposalPlease re-state the problem that we are trying to solve in this issue.Update all references from "workspace chat" to "expense chat" What is the root cause of that problem?
What changes do you think we should make in order to solve the problem?
What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?N/A - Translation update What alternative solutions did you explore? (Optional)N/A Result |
ProposalPlease re-state the problem that we are trying to solve in this issue.Update all references from "workspace chat" to "expense chat" What is the root cause of that problem?Copy update What changes do you think we should make in order to solve the problem?We need to update ALL references of "workspace chat" that users can see to "expense chat".
What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?NA What alternative solutions did you explore? (Optional)NA Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job. |
Job added to Upwork: https://www.upwork.com/jobs/~021917225274770505798 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @rojiphil ( |
@rojiphil I meant to update all instances of "workspace chat" to "expense chat" in help docs. Sorry since I linked code permalink so if you opened the markdown preview, you wouldn't see the highlighted line. You need to open Code mode. |
Triggered auto assignment to @luacmartins, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
@rojiphil, thanks for the review! However, I think we should first confirm whether we want to update the documentation at all before selecting a proposal. I'm fairly certain that this change is only intended for the translations copy. The OP seems to clearly refer to system messages, and in the documentation, I believe it makes more sense to continue calling it "Workspace chat" instead of "Expense chat. @puneetlath, could you please help confirm that? Thanks! App/docs/articles/new-expensify/connections/quickbooks-desktop/QuickBooks-Desktop-Troubleshooting.md Line 83 in 4453806
App/docs/articles/new-expensify/connections/quickbooks-desktop/QuickBooks-Desktop-Troubleshooting.md Line 123 in 4453806
|
We are not using the "workspace chat" terminology anymore, so it shouldn't be anywhere in the product or the documentation. This is now referred to as an "expense chat" for a given user. |
@luacmartins Can you assign me so I can start working on PR? |
@luacmartins Huh... This is 4 days overdue. Who can take care of this? |
❌ There was an error making the offer to @mkzie2 for the Contributor role. The BZ member will need to manually hire the contributor. |
Ah yes, sorry! I thought I had done that already. All yours. |
@luacmartins @puneetlath One thing to confirm: Do we want to update the instances of "workspace chat" in our developer/code documentation as well? For example: Lines 700 to 703 in 100d5ed
App/contributingGuides/NAVIGATION_TESTS.md Line 142 in 95ec76c
App/src/libs/actions/Report.ts Line 5096 in 3ed638b
|
I say yes, let's update it everywhere. That'll make it less likely that it accidentally sneaks back into the product. |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number:
Reproducible in staging?: Needs reproduction
Reproducible in production?: Needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @puneetlath
Slack conversation (hyperlinked to channel name): #Quality
Action Performed:
Expected Result:
All references should consistently say "expense chat" instead of "workspace chat."
Actual Result:
Removed you from xxx's approval workflow and workspace chat. Previously submitted.... whisper message displayed
Copy "Workspace chat" is not updated as "Expense chat"
Workaround:
Unknown
Platforms:
Select the officially supported platforms where the issue was reproduced:
Platforms Tested:
On which of our officially supported platforms was this issue tested:Screenshots/Videos
Add any screenshot/video evidence
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @luacmartinsThe text was updated successfully, but these errors were encountered: