Feat/daily new limit

[cashpw]

This is built on top of #82. Related: #80.

[cashpw]

On second thought, this could use some tests. I'll be away for the next two weeks but could add then when I get back.

[l3kn]

Besides some minor nitpicks this looks good.

How set are you on limiting the number of new cards per day
instead of per review session?

[cashpw]

How set are you on limiting the number of new cards per day instead of per review session?

I prefer daily based on my history with flashcard applications. However, this doesn't need to be the case for everyone. I've added org-fc-review-new-limit-schedule which allows users to set either 'day or 'session.

[l3kn]

Looks good, I like how configurable this is now.

One caveat to keep in mind is that if Emacs is restarted, the daily limit is lost.
That's fine for me as long as we add this info to the docs,
persistence for the daily count / reset day can always be added in later.

Regarding testing it should be sufficient to mock different values for the daily count / reset day
and count the new cards from a selection of (unshuffled) cards.

[cashpw]

One caveat to keep in mind is that if Emacs is restarted, the daily limit is lost. That's fine for me as long as we add this info to the docs, ...

Good point. I've updated the doc strings.

persistence for the daily count / reset day can always be added in later.

I've created #98 to track this work.

[cashpw]

Checking in. Are there any changes you'd like to see before this is merged?

[cashpw]

Checking in -- is there anything you'd like to see changed in this pull request?