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

Gh7012 receipt schedule adj #7047

Merged
merged 82 commits into from
Jul 23, 2020

Conversation

dragospodariu96
Copy link
Contributor

metas-ts and others added 30 commits July 15, 2020 14:45
TODO
* unit tests for ShipmentCandidateExporter and ShipmentScheduleAuditRepository
* MIs, ErrorMessages etc to prevent fiddling with exported shipment schedules
* dev-test;
* also see & coordinante what's missing

#7012
TODO
* unit tests for ShipmentCandidateExporter and ShipmentScheduleAuditRepository
* MIs, ErrorMessages etc to prevent fiddling with exported shipment schedules
* dev-test;
* also see & coordinante what's missing

#7012
TODO
* unit tests for ShipmentCandidateExporter
* MIs, ErrorMessages etc to prevent fiddling with exported shipment schedules
* dev-test;
* also see & coordinate what's missing

#7012
TODO
* MIs, ErrorMessages etc to prevent fiddling with exported shipment schedules
* dev-test;
* also see & coordinate what's missing

#7012
TODO
* MIs, ErrorMessages etc to prevent fiddling with exported shipment schedules
* dev-test;
* also see & coordinate what's missing

#7012
TODO
* MIs, ErrorMessages etc to prevent fiddling with exported shipment schedules
* dev-test;
* also see & coordinate what's missing

#7012
TODO
* MIs, ErrorMessages etc to prevent fiddling with exported shipment schedules
* dev-test;
* also see & coordinate what's missing

#7012
TODO
* MIs, ErrorMessages etc to prevent fiddling with exported shipment schedules
* dev-test;
* also see & coordinate what's missing

#7012
TODO
* MIs, ErrorMessages etc to prevent fiddling with exported shipment schedules
* dev-test;
* also see & coordinate what's missing

#7012
TODO
* MIs, ErrorMessages etc to prevent fiddling with exported shipment schedules
* dev-test;
* also see & coordinate what's missing

#7012
…of SpringContextHolder (#6970)

#6949
(cherry picked from commit 9d6fda6)
TODO
* MIs, ErrorMessages etc to prevent fiddling with exported shipment schedules
* dev-test;
* also see & coordinate what's missing

#7012
TODO
* MIs, ErrorMessages etc to prevent fiddling with exported shipment schedules
* dev-test;
* also see & coordinate what's missing

#7012
# Conflicts:
#	backend/de.metas.swat/de.metas.swat.base/src/main/java-gen/de/metas/inoutcandidate/model/X_M_Packageable_V.java
* Use jackson-stuff in our camel route
* Send feedback to metasfresh
* Update audit records using feedback
* convert JSON to XML
* simplify audit records in metasfresh
* Also
  * CacheInvalidateMultiRequest with better support for RepoIdAwares

  #7012
@dragospodariu96 dragospodariu96 changed the base branch from master to gh7012_shipmentschedule_rest_api July 22, 2020 12:07
Comment on lines 136 to 138
//
// Cache invalidate
// We have to do this even if invalidate=false
Copy link
Member

Choose a reason for hiding this comment

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

useless comments

* @author dp
*/

public interface IReceiptSchedulePA extends ISingletonService
Copy link
Member

Choose a reason for hiding this comment

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

(as written in the concept review, please go with the existing IReceiptScheduleDAO instead)

{
// relatively small amount of records
// => fetch and reset individually
final ImmutableSet<ReceiptScheduleId> shipmentScheduleIds = queryBL.createQueryBuilder(I_M_ReceiptSchedule.class)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
final ImmutableSet<ReceiptScheduleId> shipmentScheduleIds = queryBL.createQueryBuilder(I_M_ReceiptSchedule.class)
final ImmutableSet<ReceiptScheduleId> receiptScheduleIds = queryBL.createQueryBuilder(I_M_ReceiptSchedule.class)

Base automatically changed from gh7012_shipmentschedule_rest_api to blonde_monkey_uat July 23, 2020 06:01
dragospodariu96 and others added 7 commits July 23, 2020 10:37
- changed usage of deleted PA class to DAO in C_Order.
Co-authored-by: Petrica Nanca <petrica.nanca@metasfresh.com>
Co-authored-by: Tobias Schöneberg <tobias.schoeneberg@metasfresh.com>
Co-authored-by: TheBestPessimist <cristian@tbp.land>
Co-authored-by: Kay <kay.kostelnik@metasfresh.com>
Co-authored-by: Teo Sarca <teo.sarca@metasfresh.com>
Co-authored-by: Petrica Nanca <petrica_nanca@yahoo.com>
Co-authored-by: Peter Wyss <peter.wyss@kommunikativ.ch>
Co-authored-by: Kuba Siemiątkowski <kuba@saskla.cz>
Co-authored-by: Tobias Schöneberg <metas-ts@users.noreply.github.com>
# Conflicts:
#	backend/de.metas.swat/de.metas.swat.base/src/main/java/de/metas/inoutcandidate/exportaudit/APIExportStatus.java
#	backend/de.metas.swat/de.metas.swat.base/src/main/java/de/metas/inoutcandidate/modelvalidator/C_Order.java
@dragospodariu96 dragospodariu96 merged commit 75cac97 into blonde_monkey_uat Jul 23, 2020
@dragospodariu96 dragospodariu96 deleted the gh7012_receipt_schedule_adj branch July 23, 2020 15:01
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.

None yet

8 participants