Skip to content

Conversation

@TylerLafayette
Copy link

Added a button that allows a user to continue from where they left off. I think this would be a helpful addition to the site as I find it inconvenient to have to enter the correct number to get to where I left off.
Known issue: I think this only works with RTK1 at the moment.
Screen Shot 2019-11-15 at 5 03 33 PM

@ShavingSeagull
Copy link

I would love to see this on the site. I thought I was going blind not being able to find the button to pick up from where I left off.

@fabd
Copy link
Owner

fabd commented Jan 9, 2020

Hi @TylerLafayette really sorry I don't remember seeing a notification about this pull request. It's really not obvious >_> I saw it now due to ShavingSeagull's comment.

Indeed "continue where you left off" would be a good addition. The problem is that a user can add cards in non-sequential order.

Ideally I suppose the button would only show up for a user who has no "gaps" in their flashcard set.

I think I dealt with that conditional before -- this is the function).

In fact it looks like the function you added is the same, but it does not filter cards for the particular user (doesn't use the $userId param). Though I'm guessing you meant to get the max index # in the user's cards -- filtered by RTK1 range -- and let user proceed from that index #

I suppose that conditional works for most purposes.

I appreciate the contribution, if you can fix the $userId issue I think I can merge that, though I'll have to spend some more time on it later. I'll also probably want to refactor getHeisigProgressCount() with an option to $ignoreGaps as both functions are very similar.

edit: most important being, you both let me know of a simple little improvement we can add to the site, thanks!

@TylerLafayette
Copy link
Author

Thank you for your reply! Looking back, I'm not actually sure why I thought it was necessary to make a new function. I must have assumed the preexisting one wouldn't work. Anyways, in my latest commit, I removed the function I made and changed $this->latestKanji to use ReviewsPeer::getHeisigProgressCount instead of my custom function. Thanks again for considering this feature, I'm happy to be contributing to a site I use daily.

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.

3 participants