pagy icon indicating copy to clipboard operation
pagy copied to clipboard

update: pagy_bare_rails script: for db testing

Open benkoshy opened this issue 3 years ago • 1 comments

Why this PR?

  • The use case seems to have developed from (https://github.com/ddnexus/pagy/pull/357) and again in (https://github.com/ddnexus/pagy/issues/378). There seems to be a need to test with db, though it requires a dependency.

What's changed?

  • Add example for use with external services e.g. meilisearch. The implementation might be better done with docker-compose? That use case may present itself in future, however I am not able to do it at this second.

  • Minor cleanup

benkoshy avatar Mar 10 '22 01:03 benkoshy

brunoocasali I will make the changes you've suggested.

Your point of having a separate script for Meilisearch is noted, and warrants some thought. i thank you for suggesting it.

benkoshy avatar Mar 10 '22 12:03 benkoshy

It looks good. I will merge it as soon as the CI problem of v6 will be solved.

ddnexus avatar Dec 14 '22 02:12 ddnexus