frondeus / fvtt-syrin-control

Syrinscape Online Control plugin for FoundryVTT
MIT License
16 stars 6 forks source link

QoL request #15

Closed Berytak closed 2 years ago

Berytak commented 2 years ago

Hey there,

first of all "THIS" is the module I've wanted most. It makes handling Syrinscape a breeze. No need for a second monitor or window. Now to the Qol requests:

btw, is there a way I can show you my gratitude for this module so you can buy yourself a coffee?

Cheers Berytak

frondeus commented 2 years ago

Hi,

First of all, thanks for using this module! About your requests:

I will touch these two as soon as I finish my refactor. The 0.1.0 release was more a prototype, and now I need to clean a lot of things before adding new features and polishing UX. Otherwise, I will bury myself in spaghetti code.

Therefore it might take a while to finish this issue.

AD Btw: I... didn't expect this, and I'm flattered. I will have to think about it :)

Berytak commented 2 years ago

Take your time! There is nothing more frustrating than overlooking some stuff due to a hurry ;-)

frondeus commented 2 years ago

Hi @Berytak, I just released pre-release 0.2.0-pre2: https://github.com/frondeus/fvtt-syrin-control/releases/tag/0.2.0-pre2 It should work just fine, and you could use a link to manifest to install it in your Foundry instance if you want to test it.

I need an extra couple of days to polish everything in the codebase, prepare new documentation, README, screenshots, etc. before I release it in Foundry as a stable release.

Merry Christmas and Happy New Year!

Berytak commented 2 years ago

Hey @frondeus , sure thing that I'll test the new version, but I'm a bit puzzled how to install it. If I paste the URL into the manifest install field I get an error message. image Tried the URL with and witout the module.json. Any idea what I'm doing wrong? Do I just download it and copy the files into the module folder?

Cheers! Berytak

frondeus commented 2 years ago

Let me check it quickly in an hour , it should be working but I only tested it on local machine with V9 installed

frondeus commented 2 years ago

@Berytak please check https://github.com/frondeus/fvtt-syrin-control/releases/tag/0.2.0-pre3 - I fixed the issue, now I can download it both in V9 and 8.8

Berytak commented 2 years ago

@frondeus Can you please provide me the full link you paste into the manifest URL field? I'm still getting the same error and I guess it's some kind of user error 😏

frondeus commented 2 years ago

@Berytak Absolutely! It is a link to module.json: https://github.com/frondeus/fvtt-syrin-control/releases/download/0.2.0-pre3/module.json

Berytak commented 2 years ago

@frondeus Thank you, that worked like a charm! What I've seen so far is pretty brilliant. The icon size for the one-shots is much better and I really like it, that I can add moods to a playlist. It's just a shame, that the search button is gone. btw, I've seen that it'll load all the syrinscape icons on the one-shots for you. What do I have to do to get the same behaviour? I would guess, that it's the API driven connection and not the CSV one, am I correct?

frondeus commented 2 years ago

I am going to reintroduce search button - but as a standalone feature not depending on quick insert hack :)

frondeus commented 2 years ago

About icons - unfortunately, yes - CSV does not provide links to icons, online API does. I am closing this issue :) Happy New Year!

Berytak commented 2 years ago

Thank you for your efford! Happy New Year!