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.

Sign up for GitHub

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

Jump to bottom

chore: update UI5 from v1 to v2 #3441

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from
Draft

chore: update UI5 from v1 to v2 #3441

wants to merge 12 commits into from

Conversation

OliwiaGowor
Copy link
Contributor

Description

Changes proposed in this pull request:

  • ...
  • ...
  • ...

Related issue(s)
Closes #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

@OliwiaGowor
chore: UI5 bump packages (#3439)
e5930e3 
* chore: update packages

* chore: bump ui5-react

* fix: adjust tests to not use enzyme

* chore: fix dependencies

* fix: remove unused

* fix: remove not existing import

* chore: add compatibility package
@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/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Oct 31, 2024
OliwiaGowor and others added 11 commits November 4, 2024 08:50
@OliwiaGowor
fix: adjust render, css names and imports (#3440)
527ab33 
* fix: change render

* fix: adjust css names

* fix: change imports for Toolbar

* fix: change imports for tables

* fix: rename list comp. & badge

* fix: fix imports
@mrCherry97
chore: adjust spacing to classNames (#3442)
cd703c0 
* change some spacing to new classnames

* fix merge

* pray to god, that everything is working

* pray to god, that everything is working

* some spacing and iconEnd for ExternalLink

* some spacing and iconEnd for SA token

* add no-margin class in index.scss

* adjust the rest of the spacing

* remove unused impott

* fix stupid classNames

* adjust separatorLine to tsx and use style prop

* review changes
@OliwiaGowor
fix: Ui5 codemod adjustments (#3449)
0ff4119 
* fix: ui5 codemode adjustments

* fix: fix imports

* fix: fix table imports

* fix: fix unused
@OliwiaGowor
fix: adjust DynamicPageComponent (#3445)
7b1e4dc 
* fix: adjust dynamicPage

* fix: separate actionsBar

* fix: fix actions behavior

* fix: remove unused

* fix: fix classNames

* fix: fix empty toolbar & move banners

* fix: fix dependency

* fix: remove spacing file
@mrCherry97
fix: toast adjustments (#3462)
9c39bff
@OliwiaGowor
fix: adjust tokens (#3463)
ba9135e 
* fix: adjust tokens

* fix: adjust spacing
@OliwiaGowor
fix: adjust badges (#3464)
ce4764e 
* fix: adjust badges type names

* fix: fix type mappings

* fix: remove console.log
@OliwiaGowor
fix: adjust headers size (#3467)
c85f1bd 
* fix: add 'size' to headers

* fix: add size to headers
@mrCherry97 @OliwiaGowor
fix: defaultProps (#3469)
a565ab1 
* remove defaultProps and disableEdit prop from resourcelist

* Update src/shared/components/ResourcesList/ResourcesList.js

Co-authored-by: Oliwia Gowor <[email protected]>

---------

Co-authored-by: Oliwia Gowor <[email protected]>
@mrCherry97
fix: spinner size (#3474)
a486ff6
@OliwiaGowor
fix: theme (#3475)
c702d0c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@akucharska akucharska Awaiting requested review from akucharska akucharska will be requested when the pull request is marked ready for review akucharska is a code owner

@mrCherry97 mrCherry97 Awaiting requested review from mrCherry97 mrCherry97 will be requested when the pull request is marked ready for review mrCherry97 is a code owner

@dbadura dbadura Awaiting requested review from dbadura dbadura will be requested when the pull request is marked ready for review dbadura is a code owner

@chriskari chriskari Awaiting requested review from chriskari chriskari will be requested when the pull request is marked ready for review chriskari is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
cla: yes Indicates the PR's author has signed the CLA. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Migrate ui5-webcomponents-react from v1 to v2
3 participants
@OliwiaGowor @kyma-bot @mrCherry97