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

Upgrading React 17 and Material UI 5 of DevPortal #544

Closed
wants to merge 65 commits into from
Closed

Upgrading React 17 and Material UI 5 of DevPortal #544

wants to merge 65 commits into from

Conversation

thisaltennakoon
Copy link
Contributor

$subject

thisaltennakoon and others added 30 commits January 27, 2024 18:12
@thisaltennakoon
can use npm start
b89de57
@thisaltennakoon
Removed react-hot-loader and changed web-pack config
038b611
@wso2-jenkins-bot @thisaltennakoon
[WSO2 Release] [Jenkins #704] [Release 9.0.491] prepare for next deve…
ef4a0aa 
…lopment iteration
@thisaltennakoon
Fix REST API document styling issue
981ec81
@Tharanidk @thisaltennakoon
Fix the postman collection download issue
5e0f988
@thisaltennakoon
Ran first codemod
f78ea7f 
Changed imports
@thisaltennakoon
Ran second codemod
2ab5c58
@thisaltennakoon
Ran thired codemod
095c32b
@thisaltennakoon
Removed react-hot-loader
9f62428
@thisaltennakoon
Removed withStyles error
e0abdd5
@thisaltennakoon
Fixed loading issue when generating OAuth & APIKey tokens
3e2a05f
@thisaltennakoon
Fixed Change Password not working issue
38282f8
@thisaltennakoon
Fixed create application wizard not loading issue
7a5d4c0
@thisaltennakoon
Fixed part of the GraphQL ui error
3349160
@thisaltennakoon
Fixed graphql ui loading issue
e8cb9df
@thisaltennakoon
[working] React 18 upgrade
251a48b
@thisaltennakoon
Removed CopyToClipboard
5f01feb
@thisaltennakoon
Revert "Removed CopyToClipboard"
217c983 
This reverts commit 9d30c63.
@thisaltennakoon
Removed CopyToClipboard and added a new way
8a0497e
@thisaltennakoon
Fix styling issue in tryout consoles
38bee79
@thisaltennakoon
Fixed Log out popper issue
db0dcfe
@thisaltennakoon
Fixed Try out button size issue
1418198
@thisaltennakoon
Create package-lock.json
fb51783
@thisaltennakoon
Fixed star colours not visible issue
3cbdf05
@thisaltennakoon
Fixed tryout pop up issue
7c5993d
@thisaltennakoon
FIxed API placement issue
a234145
@thisaltennakoon
Fixed postman collection downloading issue
bd11145
@thisaltennakoon
Fixed view definition button issue
202630b
@thisaltennakoon
Fixed nested commenting issue
7fcbd7e
@thisaltennakoon
Fixed API key text field size issue
df322be
@thisaltennakoon
FIx styling issues
dd7c5ec
@thisaltennakoon
Fixed button color changes
1916909
@thisaltennakoon
Fixed styling changes
f086f07
@thisaltennakoon
Fix styling issue in Subscription & Key Generation Wizard
b8893ce
@thisaltennakoon
Partially Fix styling issue in Application info bar
ffb8030
@thisaltennakoon
Fix styling issue in Application info bar
384ac04
@thisaltennakoon
Partially Fix styling issue in Application info bar
c25ff69
@thisaltennakoon
Fix styling issue in Application info bar
0e0b4a6
@thisaltennakoon
Fixed styling issues
940c4e7
@thisaltennakoon
Fix styling issue in Subscription & Key Generation Wizard
5eb871b
@thisaltennakoon
Fix styling issue in Subscription & Key Generation Wizard
9cb9e30
@thisaltennakoon
Merge branch 'material-ui-bump-devPortal-9' of https://github.com/thi…
3f531ef 
…saltennakoon/apim-apps into material-ui-bump-devPortal-9
@thisaltennakoon
Delete package-lock.json
633f956
@thisaltennakoon
Create package-lock.json
fba6a51
@thisaltennakoon
Create package-lock.json
df65a87
@thisaltennakoon
Update package.json
905423d
@thisaltennakoon
Update package.json
ad39ef3
@thisaltennakoon
Update package.json
b385c20
@thisaltennakoon
Update package.json
e86ba26
@thisaltennakoon
Update package.json
32a04c8
@thisaltennakoon
Update package.json
b69e88e
@thisaltennakoon
Fixed build issues
0eab5fc 
of e8cb9df. Need to check this again
@thisaltennakoon
Fixed build issues
486d0c9
@CLAassistant
Copy link

CLAassistant commented Feb 13, 2024 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@tmkasun
Copy link
Contributor

tmkasun commented Feb 13, 2024

Hi @thisaltennakoon ,

Since we currently use manual build configuration for the app, With this attempt to upgrade the React version, it might be worth considering switching to Create React App CRA.
We may need to use CRACO with CRA due to our overridden configs, but transitioning to CRA would likely simplify future migrations.

@thisaltennakoon thisaltennakoon marked this pull request as draft February 16, 2024 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

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

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

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

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

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@thisaltennakoon @CLAassistant @tmkasun @tharikaGitHub @wso2-jenkins-bot @Tharanidk