Remove "conformsTo", "containedIn", and "currentLocation" of EconomicResource{Create,Update}Params (!100) · Merge requests · Value Flows / VF Schemas / vf-graphql

Open srfsh requested to merge srfsh/useless-fields into sprout 3 years ago

May 10, 2022

Remove "currentLocation" from the input type EconomicResourceCreateParams · 7ff24f7a

srfsh authored 3 years ago

Since the "currentLocation" of EconomicResources can be set by "toLocation" of
EconomicEventCreateParams, having two different ways to set it doesn't make
sense. As "toLocation" is also used for other things, it is more logical to
use that field instead.

7ff24f7a

Mar 04, 2022
- update & move link to action verbs docs to Action type description · f27bfc99
  pospi authored 3 years ago
  
  f27bfc99
- remove containedIn from EconomicResource update params as well · fb1e91f9
  pospi authored 3 years ago
  
  fb1e91f9
Feb 23, 2022

Remove "conformsTo" and "containedIn" of EconomicResourceCreateParams · 87ebfe0b

srfsh authored 3 years ago

conformsTo can be set by EconomicEventCreateParams.resourceConformsTo and
containedIn is supposed to be set only by pack and unpack (it will set it to
"null").

87ebfe0b

Remove "conformsTo", "containedIn", and "currentLocation" of EconomicResource{Create,Update}Params

Merge request reports

Activity