Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
AO-379 Add new 'lots manage' right for lot endpoints.

AO-379 Add new 'lots manage' right for lot endpoints.

    • -0
    • +4
    ./migration/20190726083236971__added_lots_manage_right.sql
  1. … 4 more files in changeset.
AO-379 Change changelog ticket link to core one and remove comments.

    • -7
    • +0
    ./migration/20190726083236971__added_lots_manage_right.sql
  1. … 1 more file in changeset.
AO-379 Add migration for lots manage right creating.

    • -0
    • +11
    ./migration/20190726083236971__added_lots_manage_right.sql
OLMIS-6402: Renamed versionId to versionNumber in referencedata
OLMIS-6402: Renamed versionId to versionNumber in referencedata
OLMIS-6402: Renamed migration

    • -27
    • +0
    ./migration/20190722091929982__rename_versionId_to_versionName.sql
    • -0
    • +27
    ./migration/20190722091929982__rename_versionId_to_versionNumber.sql
OLMIS-6402: Renamed versionId field to versionNumber

    • -1
    • +1
    ./demo-data/referencedata.facility_type_approved_products.csv
    • -1
    • +1
    ./demo-data/referencedata.orderable_identifiers.csv
    • -1
    • +1
    ./demo-data/referencedata.orderables.csv
    • -1
    • +1
    ./demo-data/referencedata.program_orderables.csv
    • -1
    • +1
    ./demo-data/referencedata.supply_partner_association_orderables.csv
    • -0
    • +27
    ./migration/20190722091929982__rename_versionId_to_versionName.sql
  1. … 38 more files in changeset.
OLMIS-6164 removed two redundant ftaps from demo data

    • -2
    • +0
    ./demo-data/referencedata.facility_type_approved_products.csv
OLMIS-6414 fixed duplicate key violation

    • -4
    • +4
    ./demo-data/referencedata.facility_type_approved_products.csv
OLMIS-6414 added missing FTAPs to demo data

    • -0
    • +5
    ./demo-data/referencedata.facility_type_approved_products.csv
  1. … 1 more file in changeset.
I have restored all of products names and codes for those we use in functional tests. http://build.openlmis.org/job/OpenLMIS-functional-tests/2307/

I have restored all of products names and codes for those we use in functional tests. http://build.openlmis.org/job/OpenLMIS-functional-tests/2307/

OLMIS-6363 Restore more product names and codes for functional tests.

    • -9
    • +9
    ./demo-data/referencedata.orderables.csv
OLMIS-6363 Restore a few product names and codes for func tests.

    • -7
    • +7
    ./demo-data/referencedata.orderables.csv
Probably functional tests will fail because of missing orderables like ARV0001, Male Condom, IPV - 5 dose,injection and some other. You can find them here: https://github.com/OpenLMIS/openlmis-func...

Probably functional tests will fail because of missing orderables like ARV0001, Male Condom, IPV - 5 dose,injection and some other. You can find them here: https://github.com/OpenLMIS/openlmis-functional-tests/search?l=Gherkin&q=product. I'm not sure what should we do - adjust tests or restore those orderables.

I have not changed products that have a connection to commodity types or trade items. I have tried to not change products we use for testing but I will ask our QA's to check this.

I have not changed products that have a connection to commodity types or trade items. I have tried to not change products we use for testing but I will ask our QA's to check this.

OLMIS-6363 Update product names and codes for demo data to be more meaningful.
OLMIS-6363 Update product names and codes for demo data to be more meaningful.
OLMIS-6363 Update product names and codes for demo data to be more meaningful.

    • -10075
    • +10075
    ./demo-data/referencedata.orderables.csv
  1. … 1 more file in changeset.
removed

removed

Yes. The table can contain a lot of data so adding columns and set default values could drop performance

Yes. The table can contain a lot of data so adding columns and set default values could drop performance

Is it because dropping and recreating is faster?

Is it because dropping and recreating is faster?

I don't think we should be settings this in the constructor. I might want to use this constructor in retrieval endpoints, for example Isn't doing this in PrePersist/Update annotated method enough?

I don't think we should be settings this in the constructor. I might want to use this constructor in retrieval endpoints, for example

Isn't doing this in PrePersist/Update annotated method enough?

OLMIS-6358: Added versioning to Facility Type Approved Product resource
OLMIS-6358: Added versioning to Facility Type Approved Product resource
In the following review, I only added a new field to the FTAP resource. I will adjust endpoints in next commits and reviews. Stay tuned.

In the following review, I only added a new field to the FTAP resource. I will adjust endpoints in next commits and reviews. Stay tuned.