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

[BUG] cannot deploy Import file Activity with destination SMS #745

Open
1 of 2 tasks
michalchalecki opened this issue Feb 15, 2023 · 7 comments · Fixed by #1209
Open
1 of 2 tasks

[BUG] cannot deploy Import file Activity with destination SMS #745

michalchalecki opened this issue Feb 15, 2023 · 7 comments · Fixed by #1209
Assignees
Labels
bug Something isn't working c/importFile COMPONENT PRIORITY used for professional service clients
Milestone

Comments

@michalchalecki
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

When deploying import activity error appears: Internal Server Error.
Import is from Data Extension to All Subscribers list
2023-02-15T15.22.02.406Z.log
ContactSMS_PPD-GL4.importFile-meta.txt

Expected Behavior

Import is successfully deployed.

Steps To Reproduce

  1. Move the ContactSMS_PPD-GL4.importFile-meta.json file to directory: deploy/MCREEV/PreProd/importFile
  2. Run mcdev deploy MCREEV/PreProd
  3. See error "Internal Server Error"

Version

4.3.3

Environment

  • OS: Windows 11
  • Node: 16.15.0
  • npm: 9.3.1

Participation

  • I am willing to submit a pull request for this issue.

Additional comments

No response

@michalchalecki michalchalecki added bug Something isn't working NEW not yet analyzed new issues get this label labels Feb 15, 2023
@JoernBerkefeld JoernBerkefeld added this to the 4.4.0 milestone Feb 15, 2023
@JoernBerkefeld JoernBerkefeld self-assigned this Feb 15, 2023
@JoernBerkefeld JoernBerkefeld added c/importFile COMPONENT PRIORITY used for professional service clients labels Feb 15, 2023
@github-project-automation github-project-automation bot moved this to Todo in mcdev Feb 15, 2023
@JoernBerkefeld JoernBerkefeld changed the title [BUG] Errror [BUG] cannot deploy Import file Activity with destination SMS Feb 15, 2023
@JoernBerkefeld
Copy link
Contributor

JoernBerkefeld commented Feb 15, 2023

Relevant Log extract:

16:22:13.018 debug: - importFile ContactSMS_PPD-GL4: Import Destination Type SMS not fully supported. Deploy might fail. 
16:22:13.177 debug: undefined 
16:22:13.177 debug: {"documentation":"https://developer.salesforce.com/docs/atlas.en-us.mc-apis.meta/mc-apis/error-handling.htm","errorcode":0,"message":"Internal Server Error"} 
16:22:13.177 error: ☇ error creating importFile ContactSMS_PPD-GL4 / ContactSMS_PPD-GL4: 
16:22:13.178 debug: Util.signalFataError() sets process.exitCode = 1 
16:22:13.178 error: • Internal Server Error 
16:22:13.178 debug: Util.signalFataError() sets process.exitCode = 1 
16:22:13.179 info: importFile upsert: 0 of 1 created / 0 of 0 updated

@JoernBerkefeld
Copy link
Contributor

the api does not return a value for the target in these cases. it should say "all contacts sms" or sth similar but it doesnt :/

@JoernBerkefeld JoernBerkefeld modified the milestones: 4.4.0, 4.4.1 Mar 10, 2023
@JoernBerkefeld JoernBerkefeld modified the milestones: 5.0.1, 5.0.2 May 5, 2023
@JoernBerkefeld JoernBerkefeld removed the NEW not yet analyzed new issues get this label label May 5, 2023
@JoernBerkefeld
Copy link
Contributor

new idea: switch to legacy API?
image

@JoernBerkefeld
Copy link
Contributor

/legacy/v1/beta/mobile/imports/ seems to get me a list of SMS import definitions - but it's lacking the field mapping and notificationEmailAddress

image

however, that can then be extended using the returned id to call this endpoint:
/legacy/v1/beta/workers/import/definition/ZFhvUUdwRFhFZW1penhRQzdKVHM4UTo0MTow
image

@JoernBerkefeld
Copy link
Contributor

JoernBerkefeld commented Jun 7, 2023

for UPSERT there seem to be multiple endpoints, all using POST:

  • /legacy/v1/beta/workers/import/definition/
  • /legacy/v1/beta/workers/import/definition/{id}/
  • /legacy/v1/beta2/workers/import/definition/
  • /legacy/v1/workers/import/definition/ OR - /legacy/v1/v1/workers/import/definition/

@JoernBerkefeld
Copy link
Contributor

figured i'd test this despite discovery not mentioning it: the above 4 upsert endpoints do NOT work with GET

@JoernBerkefeld JoernBerkefeld modified the milestones: 5.1.1, 5.1.2 Jun 27, 2023
@JoernBerkefeld JoernBerkefeld modified the milestones: 6.0.4, 6.1.0 Feb 1, 2024
@JoernBerkefeld
Copy link
Contributor

the remaining issue seems to be that we don't have a way to resolve the subscriptionDefintionId - which seems to identify the code/keyword combo that's otherwise missing in the response from /legacy/v1/beta/workers/import/definition/ZFhvUUdwRFhFZW1penhRQzdKVHM4UTo0MTow

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working c/importFile COMPONENT PRIORITY used for professional service clients
Projects
Status: Todo
2 participants