Skip to content

Generalize request handling

Daniel Gerhardt requested to merge generalize-request-handling into master

This makes request handling with logic regarding retrying and reusing in-flight requests available for PUT, PATCH and DELETE requests.

Retry logic is only implicitly used for GET requests. For other methods, it needs to be enabled explicitly.

The new request handling used for all of AbstractEntityService request-related methods. Furthermore, it is used for the most common actions which do not use the AbstractEntityService:

  • Creating comments
  • Creating content answers

The effects are noticeable when trying to perform the mentioned actions in quick succession, e.g.:

  • double clicking on the "Send" button in the comment creation dialog,
  • double clicking on the "Submit" button when answering a content.

In most cases, this should only result in a single request being sent. Still, this does not completely prevent duplicated requests. There is a time window where the response has been received but the view has not changed where additional requests can be triggered. This would require handling by the UI components.

Closes: #1168 (closed)
Related: #646 (closed)

Edited by Daniel Gerhardt

Merge request reports