Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[scr_ecto-103] Improve options/configuration described in README #104

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

theirishpenguin
Copy link

The Problem

The README does not mention options such as :allow_overflow_page_number and direct configuration such as :max_page_size. This is noted in issue #103

This PR is to (make a start at least) on correcting that.

@theirishpenguin
Copy link
Author

One check fails above with the following error...

image

Is there something I should do to correct this?

(As this is just a documentation change, I'm conscious it's probably best if I don't go messing with the actual codebase as part of this PR 😄 )

@theirishpenguin
Copy link
Author

A fellow Elixir-er kindly helped me out with the build issue at https://github.com/theirishpenguin/scrivener_ecto/pull/1/files

Looking at that PR, it seems like there is an issue building scrivener_ecto with elixir: '1.15.0' / otp: '26.0.1'. I guess that this means that an update will be needed for scrivener_ecto to keep it building on otp: '26.0.1'. I might open an issue on this repo to call this out explicitly. (Given the Low Maintenance warning on this repo, I guess you won't have time to work on this actively @drewolson - which is understandable).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants