Comictagger plugin

[mizaki]

I've updated the plugin with a couple of new options and wanted to make you are aware and make sure there is no concern about hitting the API. It uses mokkari so it's following the rate limit. All these options default to off.

• Series cover support. As there is no series image, when a series is active in the series window, it gets the full issue list for the series and tries for the issue 1 cover. (1 call to /series/id and 1 call to /issue)
• Variant cover support in the issue window. (1 call to /issue)
• Variant cover matching support. This is used when auto-tagging. Using the issue number a list of possible issues are created. For each issue a call to /issue is made. It should always be limited but say if it was vague like "Batman" and issue 1, that could be quite a few results.

I think only the last one could cause concern but maybe more for the user hitting the limiting rather than upsetting the API.

Happy to remove any if you believe it will be a problem.

[bpepple]

I've updated the plugin with a couple of new options and wanted to make you are aware and make sure there is no concern about hitting the API. It uses mokkari so it's following the rate limit. All these options default to off.

* Series cover support. As there is no series image, when a series is active in the series window, it gets the full issue list for the series and tries for the issue 1 cover. (1 call to /series/id and 1 call to /issue)

* Variant cover support in the issue window. (1 call to /issue)

I don't see a problem with these.

* Variant cover matching support. This is used when auto-tagging. Using the issue number a list of possible issues are created. For each issue a call to /issue is made. It should always be limited but say if it was vague like "Batman" and issue 1, that could be quite a few results.

I think only the last one could cause concern but maybe more for the user hitting the limiting rather than upsetting the API.

Yeah, this feature I'm a bit dubious about based on resource usage to benefit provided. Depending on the numbers of users it could potentially increase my server costs for something in my opinion isn't worth attempting to work around a user's janky comic. I'd prefer this feature to be removed, unless you guys (CT devs) want to cover any potential server costs increase. 😱

[mizaki]

No problem, I'll remove that option. And, of course, if any issues arise, you know where to shout :)

[bpepple]

Cool, Thanks for the work on this.

[mizaki]

I've added a blurb/about to the settings page and have taken the text from the web page. Are you happy with this text or would you prefer something else?

[bpepple]

@mizaki That looks fine to me.