Plugin name did not make sense before

API endpoint is relative, name should reflect that

Will be easier to add new fields

…tory

It should match the corresponding property of the serializer (which is 'service' and not 'service_select')

Not relevant anymore or duplicates

Wrong place and doesn't look great

…n plugin

Need to include all form variables listed in the options to a values dictionary

The content of the attachment is now added to the result

Now includes a check on the generated values dictionary for easy testing

This is a requirement of the plugin

The user might want to send it to just the api root of the service

Makes more sense to keep static variables dict close to where it's used

The previous size was not big enough to show all elements without scroll bars

Required for simulating a receiving service

It is sufficient to only pass the relative endpoint to the .post method

This check if redundant, as the for loop will never be executed anyway if the query set is empty

It is closely related to the serializer

This will raise a KeyError if a form variable is not available in the submitted data nor in the static variables. This is better than passing is silently

They were wrong

…e service

The instructions on how to start it is now in line with the other docker compose services

The configuration options could be saved without specifying any form variables to include. This does not make much sense. Added a minimum length of 1 for the form variables list in the serializer (Thanks to Robin for the hint)

…able

Clarifies which config

Will check the connection to the configured service

…n editor

The field 'formVariables' is read-only, so need an extra const to modify its value. Note, this didn't show up in storybook testing, only in manual functional testing

…lugin

Just as an example of what will be implemented in #4980

* Use VCR instead of mock now
* Added check on the response
* Added check on attachment encoding

* Use VCR instead of mock now
* Add test for invalid response from API endpoint

PR feedback: global config checks are not really useful for this plugin right now.

A bit more descriptive and avoids possible naming conflicts

… story

For clarity.

formData will be an empty object if we add a new object, which means the default values for the configuration fields are missing

* Add required and noManageChildProps to ServiceSelect and FormVariableSelect because they are required and need to prevent some Field props being passed down to the children
* Remove max length of FormioVariableKeyField: it is based on a text field which has no length constraints
* Remove id field from RelativeAPIEndpoint TextInput: is managed automatically
* Remove Required from JSONDumpOptions: required is the default for TypedDict
* Add default value for relative_api_endpoint: guarantees that the key is always present, which gives a strict type definition
* Remove content type headers: is set automatically when using the json kwargs
* Replace 'Included/Not Included' text with icons: makes it easier for the user to see whether it is included.