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

acquisition: send an order to a vendor #2473

Merged
merged 1 commit into from
Oct 19, 2021

Conversation

zannkukai
Copy link
Contributor

  • Adds necessary methods to create a notification of type
    acquisition order to be send to a vendor.
  • Sends only non-CANCELLED lines orders for vendor.
  • Updates order lines statuses after sending an order.

Co-Authored-by: Aly Badr aly.badr@rero.ch

Code review check list

  • Commit message template compliance.
  • Commit message without typos.
  • File names.
  • Functions names.
  • Functions docstrings.
  • Unnecessary commited files?
  • Cypress tests successful?

* Adds necessary methods to create a notification of type
acquisition order to be send to a vendor.
* Sends only non-CANCELLED lines orders for vendor.
* Updates order lines statuses after sending an order.

Co-Authored-by: Aly Badr <aly.badr@rero.ch>
@zannkukai zannkukai merged commit df2bb42 into rero:US_acquisition Oct 19, 2021
@github-actions github-actions bot added f: acquisitions Related to the acquisition module f: data migration Data migration from a legacy system or a previous version dev: DB Database structure (models.py) f: notifications labels Oct 19, 2021
@zannkukai zannkukai deleted the baa-send-acq-order branch October 19, 2021 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev: DB Database structure (models.py) f: acquisitions Related to the acquisition module f: data migration Data migration from a legacy system or a previous version f: notifications
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant