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.

Sign up for GitHub

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

Jump to bottom

Buy and Sell is string is using different case in different places #1059

Closed
naleeha opened this issue Dec 7, 2023 · 0 comments
Closed

Buy and Sell is string is using different case in different places #1059

naleeha opened this issue Dec 7, 2023 · 0 comments
Assignees
@naleeha
Labels
bug Something isn't working server

Comments

@naleeha
Copy link
Contributor

naleeha commented Dec 7, 2023

Bug Report

Steps to Reproduce:

  1. When creating basket, the basket trade has "Buy" but basket constituents has "BUY"
  2. this result in change in basket trade side not updating basket constituents sides
@naleeha naleeha added bug Something isn't working server labels Dec 7, 2023
SVC-GL-OS-SYNC pushed a commit to UBS-IB/vuu that referenced this issue Dec 7, 2023
@naleeha
finos#1059 removed the redundant sides constants and made buy and sel…
71cf561 
…l all upper case everywhere
@naleeha naleeha self-assigned this Dec 7, 2023
chrisjstevo pushed a commit that referenced this issue Dec 29, 2023
@naleeha @chrisjstevo
#1059 removed the redundant sides constants and made buy and sell all…
93489ef 
… upper case everywhere
chrisjstevo pushed a commit that referenced this issue Dec 29, 2023
@naleeha @chrisjstevo
#1059 removed the redundant sides constants and made buy and sell all…
d5e9896 
… upper case everywhere
@naleeha naleeha closed this as completed Apr 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@naleeha naleeha

Labels
bug Something isn't working server
Projects
Vuu board
Archived in project
Milestone
No milestone
Development

No branches or pull requests
1 participant
@naleeha