Skip to content

Fix(Expense flow): Show vendors option if there are vendors on host#11091

Merged
gustavlrsn merged 1 commit intomainfrom
fix/hidden-vendors-option
Mar 24, 2025
Merged

Fix(Expense flow): Show vendors option if there are vendors on host#11091
gustavlrsn merged 1 commit intomainfrom
fix/hidden-vendors-option

Conversation

@gustavlrsn
Copy link
Member

Fix issue reported on Slack: https://oficonsortium.slack.com/archives/C06K5J91675/p1741877291192939?thread_ts=1740181512.687459&cid=C06K5J91675

Description

We were only displaying the option if there were vendors previously associated with the payer account.

This PR adds a fix to always show the Vendors option if there are vendors on the host (and the API returns them according to policy and if you are an admin).

@gustavlrsn gustavlrsn requested a review from hdiniz March 14, 2025 14:50
@vercel
Copy link

vercel bot commented Mar 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
opencollective-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 14, 2025 2:55pm

@SudharakaP
Copy link
Member

@gustavlrsn : Since Henrique is on vacation can someone else review this one so that we can get this bug fixed 😉

cc: @Betree

@gustavlrsn gustavlrsn requested a review from Betree March 21, 2025 09:32
@gustavlrsn gustavlrsn merged commit 90e1d62 into main Mar 24, 2025
19 checks passed
@gustavlrsn gustavlrsn deleted the fix/hidden-vendors-option branch March 24, 2025 12:00
@SudharakaP
Copy link
Member

@gustavlrsn : Thanks a lot for taking care of this one 🥇

@SudharakaP SudharakaP moved this from In Progress to Done in Issues Raised from Support Channels Mar 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants