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

Modernize landingpage #312

Merged
merged 19 commits into from
Mar 28, 2024

Conversation

bpedersen2
Copy link
Contributor

@bpedersen2 bpedersen2 commented Nov 16, 2023

Modernize the landing page service:

  • use angular 17
  • use node 20
  • use same lint setup as on scicat frontend

Tests included/Docs Updated?

  • Included for each change/fix?
  • Passing? (Merge will not be approved unless this is checked)
  • Docs updated?
  • New packages used/requires npm install?

Extra Information/Screenshots

@bpedersen2
Copy link
Contributor Author

@bpedersen2
Copy link
Contributor Author

Should update README to reflect the new versions used

…gular

Bumps [webpack-dev-middleware](https://github.com/webpack/webpack-dev-middleware) to 6.1.2 and updates ancestor dependency [@angular-devkit/build-angular](https://github.com/angular/angular-cli). These dependencies need to be updated together.

Updates `webpack-dev-middleware` from 5.3.0 to 6.1.2
- [Release notes](https://github.com/webpack/webpack-dev-middleware/releases)
- [Changelog](https://github.com/webpack/webpack-dev-middleware/blob/v6.1.2/CHANGELOG.md)
- [Commits](webpack/webpack-dev-middleware@v5.3.0...v6.1.2)

Updates `@angular-devkit/build-angular` from 13.3.7 to 17.3.2
- [Release notes](https://github.com/angular/angular-cli/releases)
- [Changelog](https://github.com/angular/angular-cli/blob/main/CHANGELOG.md)
- [Commits](angular/angular-cli@13.3.7...17.3.2)

---
updated-dependencies:
- dependency-name: webpack-dev-middleware
  dependency-type: indirect
- dependency-name: "@angular-devkit/build-angular"
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
@bpedersen2 bpedersen2 requested a review from Junjiequan March 28, 2024 08:14
@minottic
Copy link
Contributor

I tried in another project to upgrade to angular 17 from 15 and the materials update was a mess. Does this work fine? is the rendering/style still ok?

@bpedersen2
Copy link
Contributor Author

screenshot-landingpage k8s-test frm2 tum de-2024 03 28-11_30_40

Looks OK to me in our test instance

@bpedersen2
Copy link
Contributor Author


screenshot-landingpage k8s-test frm2 tum de-2024 03 28-11_32_49

@bpedersen2 bpedersen2 merged commit aa22939 into SciCatProject:develop Mar 28, 2024
4 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