Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

resolve a sync request for projects using the shared xml backend #1771

Merged
merged 1 commit into from
Sep 13, 2023

Conversation

hahn-kev
Copy link
Collaborator

@hahn-kev hahn-kev commented Sep 13, 2023

Fixes sillsdev/LfMerge#317

Description

projects previously couldn't sync when they were using the shared xml backend, this updates the lfmerge version to resolve that issue

Checklist

  • I have labeled my PR with: bug, feature, engineering, security fix or testing
  • I have performed a self-review of my own code
  • I have reviewed the title & description of this PR which I will use as the squashed PR commit message
  • I have commented my code, particularly in hard-to-understand areas
  • I have added tests that prove my fix is effective or that my feature works
  • I have enabled auto-merge (optional)

Testing

Remove a project on hold for a sync failure with a not supported exception

@hahn-kev hahn-kev added the bug An existing problem with our app in production label Sep 13, 2023
@github-actions
Copy link

Unit Test Results

362 tests   362 ✔️  21s ⏱️
  37 suites      0 💤
    1 files        0

Results for commit 7949945.

@hahn-kev hahn-kev merged commit 2093c7c into develop Sep 13, 2023
16 of 17 checks passed
@hahn-kev hahn-kev deleted the update-lf-merge branch September 13, 2023 03:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An existing problem with our app in production
Projects
None yet
Development

Successfully merging this pull request may close these issues.

System.PlatformNotSupportedException: Named maps are not supported.
1 participant