Skip to content

Conversation

@sven1103
Copy link
Contributor

This PR extends the Products class family with new properties:

  • internal unit price
  • external unit price
  • service provider (facility)

@KochTobi KochTobi requested a review from a team July 23, 2021 06:54
@KochTobi KochTobi self-assigned this Jul 23, 2021
Copy link
Member

@KochTobi KochTobi left a comment

Choose a reason for hiding this comment

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

LGTM

Comment on lines +48 to +63

/**
* Basic product constructor.
*
* Checks that all passed arguments are not null.
*
* @param name The name of the product.
* @param description The description of what the product is about.
* @param internalUnitPrice The price in € per unit for internal customers
* @param externalUnitPrice The price in € per unit for external customers
* @param unit The product unit
* @param runningNumber Number used in conjunction with ProductType{@link life.qbic.datamodel.dtos.business.services.ProductType} to identify product
* @param serviceProvider The facility providing the service product
*
* @since 2.11.0
*/
Copy link
Member

Choose a reason for hiding this comment

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

very nice JavaDoc

@sven1103
Copy link
Contributor Author

i need to update the CL on sec

sven1103 and others added 3 commits July 23, 2021 10:42
Co-authored-by: Tobias Koch <KochTobi@users.noreply.github.com>
@sven1103 sven1103 merged commit 3c78e39 into development Jul 23, 2021
@sven1103 sven1103 deleted the feature/external-unit-price branch July 23, 2021 09:00
@KochTobi KochTobi mentioned this pull request Aug 3, 2021
KochTobi added a commit that referenced this pull request Aug 3, 2021
2.11.0 (2021-08-03)
----------------------------

**Added**

* A new enumeration for facilities ``life.qbic.datamodel.dtos.business.facilities.Facility`` (<#244>)
* New properties ``internalUnitPrice``, ``externalUnitPrice`` and ``serviceProvider`` for the ``life.qbic.datamodel.dtos.business.services.Product`` and its derivatives (<#245>)
* New properties ``totalPrice`` and ``totalDiscountPrice`` for the ``life.qbic.datamodel.dtos.business.ProductItem`` and ``totalDiscountPrice`` for the ``life.qbic.datamodel.dtos.business.Offer``
* A ``life.qbic.datamodel.dtos.business.facilities.FacilityFactory`` to get the facility for a given string representation (<#247>)

**Fixed**

* ProjectSpace names are now validated (<#249>)

**Dependencies**

**Deprecated**

* The ``unitPrice`` property in ``life.qbic.datamodel.dtos.business.services.Product``
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