Assignor: Drop the unnecessary params

[haozturk]

Impact of the new feature
Assignor

Is your feature request related to a problem? Please describe.
WMCore has dropped some legacy params from workflow dict: dmwm/WMCore#9639 We shouldn't use then in workflow assignment anymore

Describe the solution you'd like
Drop these params in workflow assignment:

custodialSubType, custodialGroup, nonCustodialSubType , autoApproveSites and nonCustodialGroup

@amaltaro please let me know if there is something that i'm missing

Describe alternatives you've considered
None

Additional context
None

[amaltaro]

I think this is all that is needed on the Unified side. Sorry for the belated heads up btw!

[haozturk]

Thanks Alan. We still have workflows being assigned w/ these params. Are they going to have problems?

[amaltaro]

If you provide a key/value that is not supported either at creation or workflow assignment, yes it will fail.

[haozturk]

since when?

I mean since when the 5 params listed above aren't supported?

[haozturk]

Oh, these are not the exact params. They should start w/ capital letters and it's AutoApproveSubscriptionSites not autoApproveSites. Darn, changing the code again