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] Adding additional custom feed overwrites the existing feed #52002

Open
1 of 8 tasks
m-natarajan opened this issue Nov 4, 2024 · 12 comments
Open
1 of 8 tasks
Assignees
Labels
AutoAssignerNewDotQuality Used to assign quality issues to engineers Bug Something is broken. Auto assigns a BugZero manager. Internal Requires API changes or must be handled by Expensify staff Overdue Weekly KSv2

Comments

@m-natarajan
Copy link

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?: need reproduction
Reproducible in production?: need reproduction
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
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: @joekaufmanexpensify
Slack conversation (hyperlinked to channel name): quality

Action Performed:

  1. Set up a Visa feed
  2. Add additional Visa feed with existing one

Expected Result:

2nd Visa card should be showing as pending

Actual Result:

Existing already added visa card is overwritten
NOTE : Issue not happening with different types of card feed

Workaround:

unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Standalone
  • Android: HybridApp
  • Android: mWeb Chrome
  • iOS: Standalone
  • iOS: HybridApp
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence
2024-11-04_16-00-46.2.mp4

View all open jobs on GitHub

@m-natarajan m-natarajan added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Nov 4, 2024
Copy link

melvin-bot bot commented Nov 4, 2024

Triggered auto assignment to @garrettmknight (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.

@m-natarajan m-natarajan added Weekly KSv2 AutoAssignerNewDotQuality Used to assign quality issues to engineers and removed Daily KSv2 labels Nov 4, 2024
Copy link

melvin-bot bot commented Nov 4, 2024

Current assignee @mountiny is eligible for the AutoAssignerNewDotQuality assigner, not assigning anyone new.

@mountiny mountiny changed the title Adding additional feed overwrites the existing feed [Workspace Feeds] Adding additional feed overwrites the existing feed Nov 4, 2024
@mountiny
Copy link
Contributor

mountiny commented Nov 4, 2024

I think we might need to pass updated name like vcf2

@m-natarajan
Copy link
Author

BZ team member, Callstack will work on this issue, please wait for them to comment before moving the issue forward.

@mountiny mountiny added the Internal Requires API changes or must be handled by Expensify staff label Nov 5, 2024
@mountiny
Copy link
Contributor

mountiny commented Nov 5, 2024

As discussed in slack here https://expensify.enterprise.slack.com/archives/C05LX9D6E07/p1730794651148409?thread_ts=1730755550.229969&channel=C05LX9D6E07&message_ts=1730794651.148409 right now we are intentionally overwriting the feed when we add a new one of the same type.

We will need to build this support into newDot. We can try to use the feedID locally and increment it and handle any errors due to existing feed with this feed id using the rbr

@mountiny
Copy link
Contributor

Still to be handled

Copy link

melvin-bot bot commented Nov 18, 2024

@garrettmknight @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

not worked on yet

@muttmuure muttmuure moved this to MEDIUM in [#whatsnext] #quality Nov 19, 2024
@mountiny
Copy link
Contributor

Weekly

@mountiny
Copy link
Contributor

This should for now be only for the custom feeds to match oldDot behaviour

@mountiny mountiny changed the title [Workspace Feeds] Adding additional feed overwrites the existing feed [Workspace Feeds] Adding additional custom feed overwrites the existing feed Nov 29, 2024
@melvin-bot melvin-bot bot added the Overdue label Dec 9, 2024
@mountiny
Copy link
Contributor

mountiny commented Dec 9, 2024

Havent got around

@melvin-bot melvin-bot bot removed the Overdue label Dec 9, 2024
@melvin-bot melvin-bot bot added the Overdue label Dec 18, 2024
@garrettmknight
Copy link
Contributor

Same as above

@melvin-bot melvin-bot bot removed the Overdue label Dec 18, 2024
@melvin-bot melvin-bot bot added the Overdue label Dec 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AutoAssignerNewDotQuality Used to assign quality issues to engineers Bug Something is broken. Auto assigns a BugZero manager. Internal Requires API changes or must be handled by Expensify staff Overdue Weekly KSv2
Development

No branches or pull requests

3 participants