Skip to content

Comments

Feature/stock count#53

Merged
RealDyllon merged 9 commits intomainfrom
feature/stock-count
Feb 18, 2023
Merged

Feature/stock count#53
RealDyllon merged 9 commits intomainfrom
feature/stock-count

Conversation

@yankai14
Copy link
Contributor

@yankai14 yankai14 commented Feb 5, 2023

Modify checkout endpoint and added cron job to implement the feature

Copy link
Contributor

@chanbakjsd chanbakjsd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we validate that the changes will not cause the stock to go into negatives?

@chanbakjsd chanbakjsd disabled auto-merge February 18, 2023 15:19
Copy link
Contributor

@RealDyllon RealDyllon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@RealDyllon RealDyllon merged commit 80229e8 into main Feb 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants