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

Show the customer/vendor balance in the select field on sales and purchases forms. #208

Closed
abouolia opened this issue Aug 14, 2023 · 5 comments
Labels
good first issue Good for newcomers Improvement Created by Linear-GitHub Sync Medium priority Created by Linear-GitHub Sync

Comments

@abouolia
Copy link
Contributor

abouolia commented Aug 14, 2023

Display the customer/vendor balance beside the name in CustomersSelect and VendorsSelect for better user experience. because many users want to see the customer/vendor balance before even clicking on the select item.

Simply can be done by changing the labelAccessor prop from code to balance since no codes in customers/vendors.

Note: Blueprint select component supports a way to display text and label (label is a text aligned to right).

Screenshot 2023-08-14 at 19 22 31
@abouolia abouolia added Improvement Created by Linear-GitHub Sync Medium priority Created by Linear-GitHub Sync good first issue Good for newcomers labels Aug 14, 2023
@asenawritescode
Copy link
Contributor

Screenshot links not working.

@abouolia
Copy link
Contributor Author

abouolia commented Jan 7, 2024

@asenawritescode fixed it and I wrote more descriptive description, thanks for letting me know we're looking forward for your contribution. :)

@abouolia abouolia changed the title [BIG-52] Show the customer/vendor balance in the select field on sales and purchases forms. Show the customer/vendor balance in the select field on sales and purchases forms. Jan 7, 2024
@asenawritescode
Copy link
Contributor

@asenawritescode fixed it and I wrote more descriptive description, thanks for letting me know we're looking forward for your contribution. :)

Unfortunately, I am stuck with the docker build, there is an issue at the migration stage because I cannot access the dB as per the error messages in the front-end.

@asenawritescode
Copy link
Contributor

@abouolia should this issue be closed ?

@abouolia
Copy link
Contributor Author

@asenawritescode Yes, we have done it and merged it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers Improvement Created by Linear-GitHub Sync Medium priority Created by Linear-GitHub Sync
Projects
None yet
Development

No branches or pull requests

2 participants