SvenWerlen / moulinette-core

FoundryVTT module which provides a set of tools for DMs for managing assets.
https://www.patreon.com/moulinette
MIT License
41 stars 10 forks source link

Use the higher access one when 2+ subs to the same Patreon are detected by Moulinette #44

Closed admiralnlson closed 2 years ago

admiralnlson commented 2 years ago

Hello,

I am subbed to the Moulinette patreon at the "Dwarf miner" tier. And I subbed to the "Mad Cartographer" creator Patreon at the "Essentials Bundle" tier which does not grant integration with Moulinette. Finally I upgraded to the "All The Things™" tier which does.

Moulinette detects this as 2 different subs, until I would imagine end of the month after which only the higher-level sub will remain: image

The issue is that the assets/scenes/.. are not showing up in Moulinette Cloud I suspect b/c of the second, lower-level one. Moulinette should in this case unlock the content and ignore the lower-level of sub which does not support integration.

I realize this is maybe not an actual moulinette-core issue and rather something to be changed in the internal logic of your webserver but I thought logging it here might be good enough.

Thanks.

SvenWerlen commented 2 years ago

Thank you for reporting this. This is a special use case, indeed. I implemented a fix for this and deployed it on my server. It is obviously difficult to me to test it with a real account but I successfully implemented a test case for it. Could you please try again and confirm if this works or not?

admiralnlson commented 2 years ago

@SvenWerlen Oh wow, that was fast! And I can confirm your fix is working perfectly also.

Thanks very much!

SvenWerlen commented 2 years ago

Great! Thank you for confirming it.