-
Notifications
You must be signed in to change notification settings - Fork 1
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
Fix for new website and add date to file names #1
base: master
Are you sure you want to change the base?
Conversation
@jbkt Thanks for the pull request. I had no idea someone else was using this. :) There are a few issues/doubts I have regarding this. The following is the output I got when I tried to run the code after applying your PR
As you can see, how we consider "page" is incorrect. When I developed the script initially, there was no index page and the base url showed the latest strip (and each following page showed one strip each. So, "page" meant "strip" then). So, pages between 1 and 5 meant, I got the latest 5 strips. However, that is not happening here. Please update the code so as to get only n number of strips (where n = pageEnd - pageStart). |
That is why I added another loop on "strips", running the command: I get: |
@jbkt |
@jbkt Any update? |
No description provided.