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

Handle _exceptions/ directory #80

Closed
lidel opened this issue Feb 15, 2021 · 0 comments · Fixed by #88
Closed

Handle _exceptions/ directory #80

lidel opened this issue Feb 15, 2021 · 0 comments · Fixed by #88
Labels
dif/medium Prior experience is likely helpful effort/days Estimated to take multiple days, but less than a week help wanted need/analysis Needs further analysis before proceeding P2 Medium: Good to have, but can wait until someone steps up snapshots issues related to snapshot creation and updates

Comments

@lidel
Copy link
Member

lidel commented Feb 15, 2021

We should look into handling exceptions for articles that have / in names or are a composite of smaller pages embedded as iframes.

Example: wikipedia_en_simple_all_maxi_2021-01.zim produces:

  • _exceptions/A%2fMain_Page which we most likely should move to Main_Page/index.html
  • caveat: due to trailing slash, all relative paths need to be adjusted with ../ prefix for each level
    • I believe existing JS will fix all paths based on relative depth, or can be adjusted to to that if needed, so no need for mutating static HTML, just move files around
@lidel lidel added help wanted P2 Medium: Good to have, but can wait until someone steps up snapshots issues related to snapshot creation and updates need/analysis Needs further analysis before proceeding dif/medium Prior experience is likely helpful effort/days Estimated to take multiple days, but less than a week labels Feb 15, 2021
@lidel lidel closed this as completed in #88 Mar 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dif/medium Prior experience is likely helpful effort/days Estimated to take multiple days, but less than a week help wanted need/analysis Needs further analysis before proceeding P2 Medium: Good to have, but can wait until someone steps up snapshots issues related to snapshot creation and updates
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant