lots of duplicated files (tokens and tiles mostly), it would be good to simply avoid that by using a common tiles dir and link directly to character tokens instead of copying them for the scene.

Spells and equipment found on characters/NPCs should be made available as Item entities, especially going to be useful in the case of an export-as-module case.

in case of multiclassing and stuff.

Would it not be feasible to have an option in the converter setup screen to "Download remote assets".

This related to #45, where you could use the --json option to download every file.

Then you would get the best of both worlds - you can convert the existing files from the zip file and if the converter finds an URL - it will download the file instead of ignoring it or replacing it with mystery-man.svg for example.

Could be a python dict issue. Use collections.OrderedDict.

From @atropos
Issue Report / Feature Request
Worlds imported from Roll20 may have names which are not url-friendly and it seems the R20 converter does not rename the world when creating it via the import process. Please consider slugifying the world name and using that web-safe name for the imported content. FVTT tries to enforce this for worlds created within the app, so it would be nice if worlds created outside the app for import can abide by the same rules.

Imported World Name: Sunless Citadel
Preferred World Name: sunless-citadel

If a character or scene name used in a os.path.join starts with a "/" or a "X:" (on windows) for example, then the previous elements are skipped. See LMoP on Windows, where "2: Cragmaw Hideout" for the scene makes all of its images saved in the world root dir instead of the scenes subdirectory.

From boovov: non Compendium items are all missing.
Weapons +1 lose their magical +1 damage
some other stuff is wrong too, check for attacks, and feats and compare character sheets.

If you have a campaign you want to sell/distribute/take parts of, it might be more interesting to have it exported as a module with compendium packs for scenes/journals/etc.. rather than have it converted into a world.

waterdeep bandit captain's scimitar is 1d6 +3, but once rolled it adds the dex so it becomes 1d6 + 3 + 3

Windows Defender is currently detecting pyinstaller generated .exe as a trojan virus. It is a false positive, but other py->exe converters might not have the issue and be worth switching to.

Get text from GM notes of a token, create handout, place said handout in the map with GM only access... though it would be a pain lol

if can't find FVTT, download the dnd5e compendiums and generate to any local folder.

we can set AC now in Foundry for token bars, so we should look that up when converting

????

paths can get pretty long with the journal directories and names and if the absolute path goes > 256, the converter should put the files into an 'other_assets' directory instead.

if enabled, change the thumb/med/max.jpg urls into original.jpg

For 'magic items' folder to be Items instead.

Use of command line tools is nice but with the ever increasing options and not everyone being tech savvy, having a GUI would go a long way into making it more usable.

(probably rewrite the GUI)

Should allow users to specify a target system. If not dnd5e, don't export character sheets. Warn about this limitation for non-dnd5e.

Some handouts are in the handouts list but don't appear in the 'journalfolder' itself. orphaned items should be shown

Request is for validation of object names to condense repeated white space to a single occurrence, and eliminate any leading or trailing white space.

Someone could have a few hundred MBs of scenes that get loaded in the sidebar when they open FVTT which can negatively impact performance.

From @atropos on Discord:

Bug Report (Suspected)
I was helping a user troubleshoot Foundry today, they had imported a world from R20 using this tool and they were having really bad performance. I got into their world and discovered that they had 50 scenes that were converted over, but each scene was using the full resolution image as the scene thumbnail. Overall it was close to 600MB of images that were being loaded as "thumbs" in the Scene sidebar.

I don't know if this is perhaps a bug with the converter, or something that went wrong in this particular user's case - but I wanted to point it out in case it slipped under the radar or is a new issue. Let me know if you need any more detail in order to look into it.

From a reply about potential CORS issues when converting a campaign:

one way to avoid this problem with CORS, is to extract the campaign.json file from the exported zip, and convert that using the "use json" option in the converter.. then all the images will be downloaded by the R20Converter app, so CORS becomes irrelevant and all images (as long as they exist) would be downloaded and converted. It just means the conversion will take even more time as it will redownload everything.

The README could be improved with this information.
I've also tested this option and found that, indeed, the files that R20Exporter could not obtain were present and properly downloaded. Even if it does take more time it could be a good fallback option to try for those who used a lot of external assets in their R20 campaigns.

With furnace module/0.3.5 release, I shouldn't need to render text/drawings into tiles but I could make them directly into drawings.

Equivalence table can be found here :
https://github.com/syl3r86/roll20npcimporter/blob/master/roll20npcimporter/roll20npcimporter.js#L1611
and Felix gave him blessing is using that table in R20Converter, so should be 'easy' to implement

If we find a graphics that doesn't match 100% but matches, let's say 95% of the size and is the only one that meets the criteria, then set it as background with a position shift.
Might be a bit tricky if the size doesn't match though, would need work through that somehow.

path separator for pack files might need to be replaced into '/' so the converted module becomes universal.

can use negative width/height, no need to use a drawing

some added "." to it, or maybe even missing entirely.

options like --disable-actors or --disable-scenes or --disable-journal

separate GM notes if a handout is owned by a character, as they could see secret section

If some assets aren't available anymore because they moved, having an option to do a mapping of old url -> new url would be pretty cool to ease conversions

Requesting a validation pass on wall-layer path objects to exclude unnecessary objects placed entirely outside the bounds of the page.

https://youtu.be/03OqKSbEk34?t=1627

FVTT has no such option. Need to flip the png itself or implement it in a module and use a flag.