New feature: Reprocess missing lookups after import #5
+219
−34
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…actoring, but working)
…querying data before reprocessing, just stores the relevant fields and values during the import process. Also some minor refactoring in ObjectTransformer and PopulateSandbox
…d added progress updates
Owner
|
Wow, first time I've ever received a Pull Request, thanks Matt. I'll take a closer look when I've got time, cheers |
Author
|
Hey, wonder if you've found any time to check this PR out? ;) |
|
Any update on this PR? @codeulike |
Author
|
Wow, I had totally forgotten about this. I think the updated code is still in my fork if you want to clone that instead... |
|
@codeulike hello! Any update regarding this PR?? |
shahrier
added a commit
to shahrier/SandboxBerry
that referenced
this pull request
Mar 13, 2022
1. Merged codeulike#5 2. Added support for "Limit" 3. Added support to save (source/target) passwords 4. Updated to .Net 4.8 5 Updated the Log4Net nuget package 6. Cleaned up the using statements
Open
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I've added a new feature to update lookup relationship fields that get missed during the initial import. For example, if Accounts are imported before Contacts, but Accounts have a Contact lookup, it can't be populated. It ended up being quite similar to what UpdateRecursiveField was already doing, so I've refactored out the code that was shared by both into separate methods as well.