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

Pmapi 1708 add uploads to campaigns #386

Merged
merged 6 commits into from
Aug 23, 2022

Conversation

jlincodes
Copy link
Contributor

@jlincodes jlincodes commented Aug 22, 2022

Fixes PMAPI-1708

Checklist

  • Up to date with main
  • All the tests are passing
    • Delete all resources created in tests
  • Prettier
  • Spectral Lint
  • npm run bundle outputs nothing suspect
  • npm run postman outputs nothing suspect

Changes

Adds uploads property to Campaigns response
uploads is an array of a single Upload object that is associated with the Campaign (similar to creatives of a Campaign with only a single Upload object)

Areas of Concern (optional)

@jlincodes jlincodes self-assigned this Aug 22, 2022
@jho44
Copy link
Contributor

jho44 commented Aug 22, 2022

@julielinter Sorry about the postman script not working. That will be resolved in this PR but it has yet to be approved. So I think you can ignore that breaking :)

@jlincodes jlincodes marked this pull request as ready for review August 22, 2022 23:41
@jlincodes jlincodes requested review from a team and sudoku-lord as code owners August 22, 2022 23:41
@jlincodes
Copy link
Contributor Author

@julielinter Sorry about the postman script not working. That will be resolved in this PR but it has yet to be approved. So I think you can ignore that breaking :)

@jho44 thank you!

description: A single-element array containing the upload object that is assocated with this campaign.
type: array
minItems: 0
maxItems: 1
Copy link
Contributor

Choose a reason for hiding this comment

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

Just want to make sure to call this out, I assume this will be bumped in the future. So we are using an array of one to keep this flexible?

Copy link
Contributor Author

@jlincodes jlincodes Aug 23, 2022

Choose a reason for hiding this comment

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

@BennyKitchell That's correct!

Copy link
Contributor

@BennyKitchell BennyKitchell left a comment

Choose a reason for hiding this comment

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

Just had one clarifying question, nothing blocking 👍

@jlincodes jlincodes merged commit f3a0418 into main Aug 23, 2022
@jlincodes jlincodes deleted the PMAPI-1708-add-uploads-to-campaigns branch August 23, 2022 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants