Blesmol / pfscf

Pathfinder Society Chronicle Filler
https://blesmol.github.io/pfscf/
MIT License
7 stars 3 forks source link

Select player count for batch create #114

Open tdhsmith opened 3 years ago

tdhsmith commented 3 years ago

With the basic batch create command, player count adjustment isn't necessary, since batch fill ignores any players with no data. However pre-filling data during batch create is very useful for values that apply to the overall event, and this ends up creating dummy players that only contain the pre-filled data. It'd be cleaner if you could simply set the player count when generating the CSV.

Blesmol commented 3 years ago

I thought about having something like this as well, but currently gave it lower priority, as I process the CSV files using Excel anyways, and deleting some of the columns therein is only a couple of clicks. However, I agree that other people might do this in a different way and might be glad for such an option.

So yes, I think the proper way is to add an additional optional flag to the batch create command to give GMs the option to specify the number of players while keeping the current number of entries as default.

Blesmol commented 3 years ago

Implementing an additional flag itself is not much work, but currently I still plan to have some proper overhaul of the whole configuration thingy, which is quite a mess at the moment. I'm planning to have a simple config interface that allows easy access both to the settings of command line flags and also to configuration stored either in the pfscf dir or in the users home dir.

Had a first look at the viper framework two weeks ago, but was not really convinced so far...