Skip to content

Migrate lf-ui-components-services to use Repository API v2#51

Merged
alexgomezlf merged 4 commits intoLaserfiche:18.xfrom
daniel-derzsi-codespring:feature/daniel-repository-v2-migration
Dec 15, 2025
Merged

Migrate lf-ui-components-services to use Repository API v2#51
alexgomezlf merged 4 commits intoLaserfiche:18.xfrom
daniel-derzsi-codespring:feature/daniel-repository-v2-migration

Conversation

@daniel-derzsi-codespring
Copy link
Copy Markdown

This pull request migrates lf-ui-components-services to use repository API v2.

  • Version bumped to 18.1.0, together with breaking change explanation in CHANGELOG.md
  • NodeJS version updated to 22
  • All packages updated to the latest version
  • All used types have been migrated to use the new types from the v2 version of the repository.

@alexgomezlf
Copy link
Copy Markdown
Contributor

alexgomezlf commented Dec 11, 2025

@daniel-derzsi-codespring Which scenarios have you tested these changes in so far? In the outlook add-in or any of the sample apps yet?

@daniel-derzsi-codespring
Copy link
Copy Markdown
Author

@daniel-derzsi-codespring Which scenarios have you tested these changes in so far? In the outlook add-in or any of the sample apps yet?

I tested the scenarios present in the Outlook add-in so far: browsing folders, browsing files, importing documents, exporting e-docs and template fields.

You can find the code at https://github.com/Laserfiche-External/outlook-add-in/pull/110 - you will have to use the feature/97-DD-repo-v2 branch and provide lf-ui-components-services on the same directory level as outlook-add-in

@daniel-derzsi-codespring
Copy link
Copy Markdown
Author

@daniel-derzsi-codespring Which scenarios have you tested these changes in so far? In the outlook add-in or any of the sample apps yet?

Also tried with https://github.com/Laserfiche/lf-sample-OAuth-SPA-react now. The React sample project is now migrated to use the new V2 API also: Laserfiche/lf-sample-OAuth-SPA-react#57

Same deal, since this isn't merged yet it's using a local build of the services.

@alexgomezlf
Copy link
Copy Markdown
Contributor

alexgomezlf commented Dec 12, 2025

Amazing. I approved this PR. The branch version of the sample app with have to be increased so I will do that when I have time. Let me know if it's blocking you.
I can't publish a preview off of your fork but once it's merged I can publish a preview soon after merge
(You can ignore the failed publish and merge. I forgot that the publish won't work off of the fork)

@daniel-derzsi-codespring daniel-derzsi-codespring force-pushed the feature/daniel-repository-v2-migration branch from 26f6468 to 685e8af Compare December 15, 2025 13:46
@daniel-derzsi-codespring
Copy link
Copy Markdown
Author

Can you please merge this PR @alexgomezlf? I don't have permission to do so unfortunately

@alexgomezlf alexgomezlf merged commit e388783 into Laserfiche:18.x Dec 15, 2025
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants