Skip to content
Snippets Groups Projects

Added the product group ID to the curator products wrester

Merged Added the product group ID to the curator products wrester
add-pgid-to-curator-products into 2.8.2.1-DEVELOMENT
1 unresolved thread
Merged Nathan Bockisch requested to merge add-pgid-to-curator-products into 2.8.2.1-DEVELOMENT
1 unresolved thread

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
shared/workspaces/alembic/versions/f2a76d224984_add_product_group_id_to_the_curator_.py 0 → 100644
15 depends_on = None
16
17 old_json = """{
18 "product_locator": "{{product_locator}}",{{#data_location}}
19 "data_location": "{{data_location}}",{{/data_location}}
20 "product_type": "{{product_type}}",{{#target_list}}
21 "target_list": ["{{target_list}}"],{{/target_list}}
22 "projectMetadata": {
23 "telescope": "{{telescope}}",
24 "projectCode": "{{projectCode}}"
25 }
26 }
27 """
28
29 new_json = """{
30 "product_group_id": "{{product_group_id}}"
  • Charlotte Hausman
    Charlotte Hausman @chausman ·
    Owner

    If the product group id is coming in from the workflow trigger, why are you returning it in the wrester too?

  • Nathan Bockisch added 1 commit

    added 1 commit

    • a6703741 - Removed product group id from wrester, added the input group locator to the metadata.json template

    Compare with previous version

  • Nathan Bockisch added 1 commit

    added 1 commit

    • 84ed6abc - Fixed mustache formatting to make new fields conditional

    Compare with previous version

  • Sam Kagan unapproved this merge request

    unapproved this merge request

  • Sam Kagan approved this merge request

    approved this merge request

  • Daniel Nemergut approved this merge request

    approved this merge request

  • Charlotte Hausman approved this merge request

    approved this merge request

  • Nathan Bockisch merged

    merged

  • Nathan Bockisch mentioned in commit 4a6e9dab

    mentioned in commit 4a6e9dab

    • Please register or sign in to reply