CompSciLauren / stardew-valley-daily-screenshot-mod

:honeybee: A Stardew Valley mod that automatically takes a screenshot of your entire farm at the start of each day
https://www.nexusmods.com/stardewvalley/mods/4779
MIT License
10 stars 9 forks source link

Allow more than 1 automatic daily screenshot #87

Closed CompSciLauren closed 2 weeks ago

CompSciLauren commented 6 months ago

Is your feature request related to a problem? Please describe. It would be nice to be able to setup more than 1 daily automatic screenshot. Currently, you can only take one a day automatically, and any additional ones have to be taken manually.

The example use case from the user is to take an automatic screenshot of both the inside of the farm house and of the farm outside.

Describe the solution you'd like Allow any number of automatic screenshots, so that you can have multiple config rulesets, and any that are set to be automatic will take the screenshot.

Note that it would still take just one automatic screenshot when you first enter the specified location, and not continue to take a screenshot if you re-enter that same location later. (existing behavior, but just stating it for clarification)

Describe alternatives you've considered If not implemented, users can alternatively setup 1 automatic screenshot configuration and an additional manual one. This means they would need to remember to take the screenshot manually, though.

Additional context This was requested in a comment on Nexus on 02/04/2024 here: https://www.nexusmods.com/stardewvalley/mods/4779/?tab=posts

CompSciLauren commented 2 weeks ago

Went to implement this feature and realized it can be done with this config.json example:

{
  "AuditoryEffects": true,
  "VisualEffects": true,
  "ScreenshotNotifications": true,
  "SnapshotRules": [
    {
      "Name": "Farm Picture",
      "ZoomLevel": 0.25,
      "Directory": "Default",
      "FileName": "Default",
      "Trigger": {
        "Days": "Daily",
        "Weather": "Any",
        "Location": "Farm",
        "Key": "None",
        "StartTime": 600,
        "EndTime": 2600
      }
    },
    {
      "Name": "Farmhouse Picture",
      "ZoomLevel": 0.25,
      "Directory": "Default",
      "FileName": "Default",
      "Trigger": {
        "Days": "Daily",
        "Weather": "Any",
        "Location": "Farmhouse",
        "Key": "None",
        "StartTime": 600,
        "EndTime": 2600
      }
    }
  ]
}

Not sure why this wasn't caught before, may have setup the config.json file incorrectly when first testing whether it was possible to take multiple automatic screenshots.

Responded to original post on Nexus, closing this ticket.