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

Load latest issue image preview from pdf.stuyspec.com #43

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

Conversation

madeline-lei
Copy link
Contributor

Find the volume and issue number through the HTML text of pdf.stuyspec.com. Then, convert it into Uint8 arrays and then base64 to create temporary image links to be sent and loaded.

  • The images take some time to load, but I am unsure of how to fix that
  • Getting the volume/issue number only works if the latest one is at the bottom (but above the 9/11 one for volume)
  • However, for the issue number I had to fetch the second to last article because of the IsraelHamasWar issue, which might be a problem for future volumes (but should be an easy fix, just replace the -4 with -3)
  • Location of the file may strange since I just kept it in the old location, feel free to change

Find the volume and issue number through the HTML text of
pdf.stuyspec.com. Then, convert it into Uint8 arrays and then base64 to
create temporary image links to be sent and loaded.
  - The images take some time to load, but I am unsure of how to fix
    that
  - Getting the volume/issue number only works if the latest one is at
    the bottom (but above the 9/11 one for volume)
  - However, for the issue number I had to fetch the second to last
    article because of the IsraelHamasWar issue, which might be a
    problem for future volumes (but should be an easy fix, just replace
    the -4 with -3)
  - Location of the file may strange since I just kept it in the old
    location, feel free to change
@leomet07 leomet07 self-requested a review November 14, 2023 23:00
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.

1 participant