Uploaded image for project: 'AdMaxLocal'
  1. AdMaxLocal
  2. AML-3740

ADS accepts budget by distribution

    Details

    • Type: Task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: Mercury
    • Labels:
      None

      Description

      ADS currently has one budget for the customer. We need to have one budget per distribution. This will require changes in

      • aml-core (change SOAP endpoint)
      • ads-soap-client
      • DeploymentServiceHelper in aml-sapi

      Details regarding changes in ads-soap-client and aml-sapi:

      We need replace the current ads-soap-client Customer.java file with the new one from aml-core and and update the aml-sapi code (and related unit tests).

      Here are the commits related to the budgets. In the aml-sapi repo, the totalBudget variable goes away and instead of calling ret.setBudget() it will look something like

      DeploymentServiceHelper.java
      - ret.setBudget(totalBudget);
      + for (Service service : serviceBudgetMap.keySet()) {
      +     ret.getBudgetMap().setBudget(mapDistribution(service), serviceBudgetMap.get(service)).
      + }
      

      https://bitbucket.org/tsa/aml-sapi/commits/e886f60f8d78705d851db34b83a76ba54a21d2c5

      ads-soap-client needs the new Customer.java so it can correctly serialize the object to XML.

      https://bitbucket.org/tsa/ads-soap-client/commits/b1a2ced56149dd07c9f33fd2dcbdea665924558d#chg-src/main/java/com/thesearchagency/core/smb/service/admaxdeployment/model/CampaignDeploymentSO.java

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                varshini.padmanabhan Varshini Padmanabhan (Inactive)
                Reporter:
                therouxj Jeff Theroux
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: