Jumoo / uSync.Complete.Issues

Public Issue tracker and roadmap for uSync.Complete
https://jumoo.co.uk/usync/complete/
2 stars 1 forks source link

Syncing user groups not working (as expected) #240

Open karltynan opened 2 weeks ago

karltynan commented 2 weeks ago

Describe the bug

User groups are not syncing to disc on save or export.

Clicking "Report" or "Export" shows as Done in Umbraco and no files are generated.

This could be user error due to not understanding the documentation, but I have uSync, uSync Complete AND uSync People Edition installed - it wasn't clear if I needed to install uSync and uSync People Edition for uSync Complete.

uSync settings: appsettings.json

"uSync": {
    "People": {
        "EnabledDefault": true
    },
    "Publisher": {
        ...
    },
    "Settings": {
        "ImportAtStartup": "Settings",
        "ExportOnSave": "None",
        "UIEnabledGroups": "Settings,Users"
    },
    "Sets": {
        "Default": {
            "Handlers": {
                "DictionaryHandler": {
                    "Group": "Settings",
                    "Settings": {}
                }
            }
        }
    }
}

uSync settings: appsettings.Development.json

"uSync": {
    "Settings": {
        "ImportAtStartup": "None",
        "ExportOnSave": "Settings,Users"
    }
}

Versions

To Reproduce

  1. Install the noted packages in Umbraco
  2. Try to export user groups to disc, or
  3. Try to save a change to user groups

Expected behaviour

uSync would generate the files to be able to sync user groups between environments on save or export, much like content, settings etc.

Actual behaviour

No files are generated on save or export. uSync reports as Done when trying to "Report" or "Export" user groups. Neither approach generates any sync files.

Screenshots

Backoffice reporting:

image

Backoffice settings:

image

Solution files:

image

Many thanks in advance 😃

KevinJump commented 2 weeks ago

Yes.

Looks like we updated some code and forgot to account for the fact the the default user is -1 and the root of the tree is also -1. 🤦

fixed now.

nightly release : https://dev.azure.com/jumoo/Public/_artifacts/feed/nightly/NuGet/uSync.Complete/overview/13.1.6-build.20240830.1

will package up next week, there are some other fixes in this. so we need to release note it up.

Kevin

karltynan commented 2 weeks ago

Brilliant, thanks!

karltynan commented 1 week ago

Out of curiosity, do I need to install uSync and uSync People Edition as well as uSync Complete?

KevinJump commented 1 week ago

Hi, you should only need to install the usync.Complete package as that has dependencies on the other two (so they will be added automatically).