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

fix: Ui5 codemod adjustments #3449

Merged

Conversation

OliwiaGowor
Copy link
Contributor

@OliwiaGowor OliwiaGowor commented Nov 4, 2024

Description

Changes proposed in this pull request:

  • adjustments made with UI5 Web Components for React codemod

Related issue(s)
#3422

Definition of done

  • The PR's title starts with one of the following prefixes:
    • feat: A new feature
    • fix: A bug fix
    • docs: Documentation only changes
    • refactor: A code change that neither fixes a bug nor adds a feature
    • test: Adding tests
    • chore: Maintainance changes to the build process or auxiliary tools, libraries, workflows, etc.
  • Related issues are linked. To link internal trackers, use the issue IDs like backlog#4567
  • Explain clearly why you created the PR and what changes it introduces
  • All necessary steps are delivered, for example, tests, documentation, merging

@kyma-bot kyma-bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cla: yes Indicates the PR's author has signed the CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Nov 4, 2024
@OliwiaGowor OliwiaGowor marked this pull request as ready for review November 4, 2024 18:05
@kyma-bot kyma-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 4, 2024
Copy link
Contributor

@mrCherry97 mrCherry97 left a comment

Choose a reason for hiding this comment

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

Check warning on line 23 in src/components/Clusters/views/ClusterOverview/ClusterValidation/ClusterValidation.tsx
'Loader' is defined but never used

What about badges in extensibility? Are changes with warning -> critical affecting anything?

@@ -20,6 +20,8 @@ import { isResourceEditedState } from 'state/resourceEditedAtom';
import { isFormOpenState } from 'state/formOpenAtom';
import { handleActionIfFormOpen } from '../UnsavedMessageBox/helpers';

import { ObjectPageHeader } from '@ui5/webcomponents-react';
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
import { ObjectPageHeader } from '@ui5/webcomponents-react';

@OliwiaGowor
Copy link
Contributor Author

For the badges, there needs to be done whole badges component adjustment because of those changes. I would propose to do it in separate PR. The small change in this PR won't affect extensibility.

@kyma-bot kyma-bot added the lgtm Looks good to me! label Nov 6, 2024
@mrCherry97 mrCherry97 merged commit 0ff4119 into kyma-project:ui5-migrate-v2 Nov 6, 2024
6 of 13 checks passed
@OliwiaGowor OliwiaGowor deleted the ui5-codemode-adjustments branch November 8, 2024 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Indicates the PR's author has signed the CLA. lgtm Looks good to me! size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants