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

[Workspace Feeds] [$50] Change Cards & Domains tab to Domains in settings #45021

Closed
kevinksullivan opened this issue Jul 8, 2024 · 61 comments
Closed
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Design External Added to denote the issue can be worked on by a contributor NewFeature Something to build that is a new item. Reviewing Has a PR in review Weekly KSv2

Comments

@kevinksullivan
Copy link
Contributor

kevinksullivan commented Jul 8, 2024

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Reproducible in staging?: Y
Reproducible in production?: Y

Slack conversation: https://expensify.slack.com/archives/C06ML6X0W9L/p1720038761870689

Action Performed:

  1. Sign up on NewDot
  2. Create a workspace
  3. Navigate to settings

Expected Result:

Cards & Domains tab should just say Domains, since cards will be supported in NewDot within the workspace.

Actual Result:

The tab says Cards & Domains.

Screenshots/Videos

image

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01b56ebd1169d98c19
  • Upwork Job ID: 1816676250720638908
  • Last Price Increase: 2024-08-08
Issue OwnerCurrent Issue Owner: @DylanDylann
@kevinksullivan kevinksullivan added Daily KSv2 NewFeature Something to build that is a new item. Bug Something is broken. Auto assigns a BugZero manager. labels Jul 8, 2024
Copy link

melvin-bot bot commented Jul 8, 2024

Triggered auto assignment to @isabelastisser (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details. Please add this Feature request to a GH project, as outlined in the SO.

Copy link

melvin-bot bot commented Jul 8, 2024

Triggered auto assignment to @anmurali (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Daily KSv2 labels Jul 8, 2024
Copy link

melvin-bot bot commented Jul 8, 2024

⚠️ It looks like this issue is labelled as a New Feature but not tied to any GitHub Project. Keep in mind that all new features should be tied to GitHub Projects in order to properly track external CAP software time ⚠️

Copy link

melvin-bot bot commented Jul 8, 2024

Triggered auto assignment to Design team member for new feature review - @dubielzyk-expensify (NewFeature)

@kevinksullivan
Copy link
Contributor Author

also discussing here

@kevinksullivan
Copy link
Contributor Author

No UI changes need so I will take @dubielzyk-expensify off of this.

Copy link

melvin-bot bot commented Jul 22, 2024

@anmurali @isabelastisser @mountiny this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@mountiny
Copy link
Contributor

all good Melvin

@kevinksullivan kevinksullivan changed the title [Held on workspace feeds] Change Cards & Domains tab to Domains in settings Change Cards & Domains tab to Domains in settings Jul 26, 2024
@kevinksullivan kevinksullivan added the External Added to denote the issue can be worked on by a contributor label Jul 26, 2024
Copy link

melvin-bot bot commented Jul 26, 2024

Job added to Upwork: https://www.upwork.com/jobs/~01b56ebd1169d98c19

@melvin-bot melvin-bot bot changed the title Change Cards & Domains tab to Domains in settings [$250] Change Cards & Domains tab to Domains in settings Jul 26, 2024
@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Jul 26, 2024
Copy link

melvin-bot bot commented Jul 26, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @DylanDylann (External)

@kevinksullivan kevinksullivan removed the Monthly KSv2 label Jul 26, 2024
@JmillsExpensify
Copy link

Said differently, I think we should make this change now. Cards and Domains is inaccurate so we should fix this ASAP. The feature someone is accessing is domains/domain control. Let's make that clear and change this to Domains now, not later.

@trjExpensify
Copy link
Contributor

We're confusing lots of users who are looking for personal card import.

Are they? I just know this convo came up on the topic of (coming soon) workspace feed card features in the workspace. At which point, it's confusing to point to card management elsewhere with "Cards & Domains". Until then, we don't have any visibility for card settings.

Not passionate, we're talking a week or so until we take it out of beta.

@JmillsExpensify
Copy link

JmillsExpensify commented Aug 8, 2024

Right, but I saying that we don't have card functionality on NewDot either way. Again, looking at Fullstory sessions, this is consistently a point of confusion. People click this button and immediately come back and start clicking somewhere else. They are looking for something that we don't have in my opinion, which is personal card import.

@mountiny
Copy link
Contributor

mountiny commented Aug 8, 2024

Created a draft for this, just simple copy changes #47119

@JmillsExpensify
Copy link

JmillsExpensify commented Aug 8, 2024

As a result, I don't think this is a solution to anything in particular, so I think we should remove it. Whether or not we have workspace feeds doesn't change anything for me. You still need to create a workspace and then go to More Features to find company feeds regardless - whenever that exists.

@trjExpensify
Copy link
Contributor

They are looking for something that we don't have in my opinion, which is personal card import.

Alright, or they're just early exploring the features and not ready to connect cards yet.

Created a draft for this, just simple copy changes #47119

Ship it 👍

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Aug 8, 2024
@mountiny
Copy link
Contributor

mountiny commented Aug 8, 2024

Shipped

@nkdengineer sorry this was just quick fix

Copy link

melvin-bot bot commented Aug 8, 2024

Triggered auto assignment to @shawnborton (Design), see these Stack Overflow questions for more details.

@mountiny
Copy link
Contributor

mountiny commented Aug 8, 2024

@shawnborton wanted to ask if we should also update the icon for the Domains now that we only link to domains and no cards

image

@dubielzyk-expensify
Copy link
Contributor

I like the idea of using the globe icon only here, but I'll let Mr Borton have the call.
image

@JmillsExpensify
Copy link

Agree with the global icon only, though I don't have approval power over Mr. Borton.

@mountiny
Copy link
Contributor

mountiny commented Aug 8, 2024

I can use the Globe icon, I am not Borton though so I am worried to see what might happen

@mountiny
Copy link
Contributor

mountiny commented Aug 8, 2024

image

@JmillsExpensify
Copy link

Maybe we push the button and if Mr Borton is mad tomorrow we can re-assess.

@trjExpensify
Copy link
Contributor

See ya'll in detention!

@shawnborton
Copy link
Contributor

Hi there, Mr. Borton here, and I like where you all landed. Class dismissed.

@mountiny
Copy link
Contributor

All good to be closed here

@DylanDylann
Copy link
Contributor

@anmurali @mountiny It seems we missed to process payment on this issue

@mountiny
Copy link
Contributor

mountiny commented Oct 2, 2024

@DylanDylann, can you please remind for what the payment is? I only see internal PR with internal review

@DylanDylann
Copy link
Contributor

@mountiny
Copy link
Contributor

mountiny commented Oct 2, 2024

I think this was one line copy change and there was no pr either. sorry that there was miscommunication about if this should be external or not, but I think we should just move ahead for this one as it was not complex task

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Design External Added to denote the issue can be worked on by a contributor NewFeature Something to build that is a new item. Reviewing Has a PR in review Weekly KSv2
Projects
Status: Done
Development

No branches or pull requests