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

partial implementation of #949 and #930 enhancement adjustments #951

Merged
merged 3 commits into from
May 17, 2021
Merged

partial implementation of #949 and #930 enhancement adjustments #951

merged 3 commits into from
May 17, 2021

Conversation

bluerthanever
Copy link
Contributor

  1. removed some codes from implementation full sized cover image enhancement
  2. partial implementation of MEGA link @Fanbox #949, to store urls found in content that are not images, and calls PixivHelper.write_url_in_description to write urls to file.
  3. updated test cases.

1. removed some codes from implementation full sized cover image enhancement
2. added `descriptionUrlList` for `FanboxPost` to use `PixivHelper.write_url_in_description` to write urls
    i just added urls that could be found in html body which are not images, p blocks, and embeds.
calls `PixivHelper.write_url_in_description` after processing all images, writing information and htmls
@bluerthanever
Copy link
Contributor Author

i changed some codes from 930 implementation because i think some are not so necessary, for example the new field for FanboxPost which in codes is actually set the same as coverImageUrl, or you could say the other way around...
also before the implementation, cover images would be added to embedded files, but you seem to have removed that. so i added that back...

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.

2 participants