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

Import updates #54

Closed
wants to merge 10 commits into from
Closed

Import updates #54

wants to merge 10 commits into from

Conversation

solocommand
Copy link
Collaborator

@solocommand solocommand commented Nov 3, 2016

Update import logic to account for changes made in #39

  • Set brand info for Omeda integration

Content User Rel import

  • Import RMI content_user_rel entries as input-submission and input-answer
    • Omeda questions omeda_5070622
    • Non-omeda question GROUP_industry
    • Omeda non-standard questions fcp_business
    • Comments
      • Support retrieving question map per group
    • Purchase Intent

Resulting diff of newest 3 account, external, and internal identities: https://www.diffchecker.com/teO5w0Bx

@zarathustra323
Copy link
Member

@solocommand This is still pending changes, correct?

@solocommand
Copy link
Collaborator Author

@zarathustra323 Yes

@solocommand
Copy link
Collaborator Author

solocommand commented Nov 7, 2016

@zarathustra323 Actually it can be merged as-is and my next set of changes separated out.

@solocommand
Copy link
Collaborator Author

@zarathustra323 Excluding non-standard omeda questions for other groups (fcp_industry/business) and other inquiry types, this is ready for review

@zarathustra323
Copy link
Member

@solocommand The stringified MongoId value for a phones object is not being set. It looks like you're doing it for emails and addresses, but phones was excluded.

…pdates

# Conflicts:
#	src/AppBundle/Controller/SandboxController.php
@solocommand solocommand closed this Jan 4, 2017
@solocommand solocommand mentioned this pull request Jan 4, 2017
10 tasks
@solocommand solocommand deleted the import-updates branch April 12, 2017 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants