If a mapping is set up on a system after data already exists for an entity, all entities must be re-saved to trigger a push of those entities. Would be helpful to just have a drush command to do this.

CommentFileSizeAuthor
#7 salesforce_push_drush-2903071-7.patch3.11 KBgcb
#4 salesforce_push_drush-2903071-4.patch3.24 KBgcb
#2 salesforce_push_drush_2903071-1.patch2.53 KBgcb

Comments

gcb created an issue. See original summary.

gcb’s picture

gcb
he/him
Location Portland, OR
commented
StatusFileSize
new salesforce_push_drush_2903071-1.patch2.53 KB

Status: Needs review » Needs work

The last submitted patch, 2: salesforce_push_drush_2903071-1.patch, failed testing. View results

gcb’s picture

gcb
he/him
Location Portland, OR
commented
StatusFileSize
new salesforce_push_drush-2903071-4.patch3.24 KB

Re-rolled against latest dev. Includes some code clean-up.

gcb’s picture

gcb
he/him
Location Portland, OR
commented
Assigned: Unassigned » aaronbauman
Status: Needs work » Needs review
aaronbauman’s picture

aaronbauman
he/him
commented
Status: Needs review » Needs work

Looks good, with minor adjustments in the command declaration:

+  // Deletes the temporary node table column created by save-moderation-states.

get rid of copy-paste cruft.

+Arguments:
+ mapping_id                                 The machine name of the mapping for the entities.
+ count                                      The number of entities to try to sync. (Default is 50).",
  • "arguments" should be a key, and its value should be an array.
    Each entry should be key-value pair with the name of an argument and its description.
    e.g. 'arguments' => [ 'mapping_id' => 'The machine name of the mapping...' ],
  • "count" should be in an "options" array, since it's not required and has a default.
gcb’s picture

gcb
he/him
Location Portland, OR
commented
Status: Needs work » Needs review
StatusFileSize
new salesforce_push_drush-2903071-7.patch3.11 KB

How's this look?

aaronbauman’s picture

aaronbauman
he/him
commented
Status: Needs review » Fixed

This is in with minor changes.
Thanks!

  • aaronbauman committed e6324a9 on 8.x-3.x authored by gcb 
    Issue #2903071 by gcb, aaronbauman: Add ability to trigger push on...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.