Looking to get some advice on passing arguments to constructors

Absolutely correct (assuming this->action is a UObject). Though it would be closer to the following:

this->action = NewObject<WhateverClassActionIs>(WhateverClassActionIs::StaticClass());
this->action->Init(target);