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

[AAE-20109] Move alfresco js-API and alfrescoapi service out from the core #9317

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

eromano
Copy link
Contributor

@eromano eromano commented Feb 9, 2024

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (check one with "x")

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation
  • Other... Please describe:

What is the current behaviour? (You can also link to an open issue here)
https://hyland.atlassian.net/browse/AAE-20109

What is the new behaviour?

Does this PR introduce a breaking change? (check one with "x")

  • Yes
  • No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...

Other information:

@BSekula
Copy link
Contributor

BSekula commented Feb 27, 2024

@eromano I rebased the PR

@BSekula BSekula force-pushed the dev-eromano-AAE-12502 branch 3 times, most recently from 2cfe5ee to b227537 Compare February 29, 2024 13:10
@BSekula BSekula force-pushed the dev-eromano-AAE-12502 branch 2 times, most recently from 176ea94 to 1c6a364 Compare March 7, 2024 14:22
@BSekula
Copy link
Contributor

BSekula commented Mar 7, 2024

Added migration schematics for adf-core@7.0.0

Steps how to test it:

  1. Checkout this branch
  2. Run npm link in lib/core
  3. Go to your project where you would like to perform the migration
  4. Run npm link @alfresco/adf-core
  5. Run command ng update @alfresco/adf-core --from 6 --to 7 --migrate-only

In case you got an error with versions, please change version in angular.json to 1

@BSekula BSekula force-pushed the dev-eromano-AAE-12502 branch 3 times, most recently from 849f6ac to 6d30a95 Compare March 21, 2024 09:47
@BSekula BSekula force-pushed the dev-eromano-AAE-12502 branch 4 times, most recently from 1f3cd6c to f18ba12 Compare March 28, 2024 10:43
@BSekula BSekula force-pushed the dev-eromano-AAE-12502 branch 3 times, most recently from b47d6ef to 899dbfb Compare April 9, 2024 13:16
@BSekula BSekula force-pushed the dev-eromano-AAE-12502 branch 3 times, most recently from 8d4ae82 to aee39f2 Compare April 15, 2024 15:59
@BSekula BSekula force-pushed the dev-eromano-AAE-12502 branch 8 times, most recently from f386255 to 0213487 Compare May 22, 2024 09:37
@BSekula BSekula force-pushed the dev-eromano-AAE-12502 branch 3 times, most recently from 2b2b3d9 to 54097e1 Compare May 23, 2024 10:15
run: |
set -u;
./scripts/update-version.sh -gnu || exit 1;
- name: Set migrations
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here we are setting migration version to e.g. 7.9.0-1234567890 (current release version) in lib/core/schematics/migrations/collection.json

"schematics": {
"move-out-alfresco-api": {
"description": "Update alfresco-api imports",
"version": "7.0.0",
Copy link
Contributor

@BSekula BSekula May 23, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Versions are going to be changed by CI to current release version, so repos can be updated it like:

npx nx@17.0.2 migrate @alfresco/adf-core@6.10.0-9192302350

{
name: 'move-out-alfresco-api',
allowedVersionScope: '<=7.0.0'
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here we are setting migration version to current released version e.g. 6.9.0-1234566, until it satisfy the allowedVersionCope

@BSekula BSekula force-pushed the dev-eromano-AAE-12502 branch 2 times, most recently from ffac124 to c4df262 Compare May 23, 2024 12:18
Copy link

sonarcloud bot commented May 23, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
5.2% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants