renderedtext / render_async

render_async lets you include pages asynchronously with AJAX
https://rubygems.org/gems/render_async/
MIT License
1.08k stars 75 forks source link

Add support for retry count header #147

Closed reneklacan closed 3 years ago

reneklacan commented 3 years ago

Currently there's no way to tell on backend if request is being retried or not. This change adds support for optional retry_count_header parameter which configures request to include retry count in specified header. (This is for example useful when you for example know that previous request timed out and you want to reduce workload to try to make it on time)

This change also fixes retry_count which is currently broken if retry_delay is not specified.

reneklacan commented 3 years ago

To address integration tests failure I created PR to update base app https://github.com/nikolalsvk/rails-6-base-app/pull/29

nikolalsvk commented 3 years ago

It's released in 2.1.11 here https://github.com/renderedtext/render_async/releases/tag/2.1.11, thank you 🙇

reneklacan commented 3 years ago

@nikolalsvk awesome, thanks! :tada: