H2Owater425 / node-hitomi

Hitomi.la api for Node.js
https://npm.im/node-hitomi
MIT License
15 stars 2 forks source link

fix: make method to get image url flexible(#36) #45

Closed bass9030 closed 2 years ago

bass9030 commented 2 years ago

As a result of trying all 25 recently uploaded galleries, it worked normally! 최근 업로드된 25개의 작품들을 시도해본 결과, 정상작동했습니다!

H2Owater425 commented 2 years ago

First of all, thank you for making the pull request! I felt that we had the similar idea after reading your code But unfortunately, this package aims to be dependency-free to get benefits on size, browser support, and lots more (That's why I issued #15 and implemented the fetchBuffer function a long time ago. Plus, the sync-request that is used in modified code is lack performance and is unstable) And I hope users obtain updated data while running their code, rather than restarting them all (And I'm implementing this) Originally, I was planning to release a new major version including text search, refactoring, flexible image url resolver, but I'll move up the schedule excluding the text search function (you'll soon find the new version that supports text search, you don't have to worry about it) Anyway, thank you so much for your contribution In condlusion, I was unable to pull this request due to difficulties in applying the code, but thank you for contribution

우선, 풀 리퀘스트를 해주셔서 감사합니다! 코드를 읽어보니 저와 비슷한 생각을 하고 계신다는 것을 느꼈습니다 하지만, 이 패키지는 크기, 브라우저 지원과 같은 여러 이점을 위해 종속성으로부터의 자유를 표방하고 있습니다 (이슈 #15를 생성하고 fetchBuffer 함수를 구현한 것도 이 때문입니다. 또한, 수정된 코드에 사용된 sync-request 패키지는 성능이 좋지 않으며 불안정합니다) 그리고, 사용자들이 코드를 재실행하는 것보다는 실행 중에 변경된 데이터를 받아올 수 있기를 바랍니다 (이를 구현 중에 있습니다) 원래는 문장 검색, 리팩토링, 유연한 이미지 생성기를 포함한 새 주 버전을 출시하려고 했지만, 문장 검색 기능을 제외하고 일정을 앞당기기로 했습니다 (문장 검색을 지원하는 새로운 버전은 금방 출시될 예정이니 걱정하지 않으셔도 됩니다) 결과적으로 수정 사항을 반영하지는 못하게 되었으나, 진심으로 기여에 감사드립니다