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

[Spring Cleanup] Remove Spring Framework Dependency #805

Merged
merged 9 commits into from
Jan 24, 2025
Merged

[Spring Cleanup] Remove Spring Framework Dependency #805

merged 9 commits into from
Jan 24, 2025

Conversation

lashinijay
Copy link
Contributor

Purpose

This PR removes dependencies and usages of the Spring framework in the following REST APIs.

  • Application management API
  • Userstore management API
  • Self service management API

Related Issue(s)

ThaminduR
ThaminduR reviewed Jan 24, 2025
View reviewed changes
...on/identity/api/server/application/management/common/ApplicationManagementServiceHolder.java
}
return service;

} catch (NullPointerException e) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this needed?

@lashinijay lashinijay force-pushed the master-remove-spring branch 2 times, most recently from 116667d to 01e5a5f Compare January 24, 2025 16:42
ThaminduR
ThaminduR previously approved these changes Jan 24, 2025
View reviewed changes
@lashinijay lashinijay force-pushed the master-remove-spring branch from 01e5a5f to ddbc1d9 Compare January 24, 2025 17:17
@lashinijay
Copy link
Contributor Author

Self Service API

Screenshot 2025-01-24 at 22 46 12

Userstore API

Screenshot 2025-01-24 at 22 31 45

Application API

Screenshot 2025-01-24 at 22 29 26

@lashinijay lashinijay force-pushed the master-remove-spring branch from ddbc1d9 to 40050d4 Compare January 24, 2025 17:20
ThaminduR
ThaminduR previously approved these changes Jan 24, 2025
View reviewed changes
ThaminduR
ThaminduR previously approved these changes Jan 24, 2025
View reviewed changes
@lashinijay @ThaminduR
fix license header
390f3dc 
Co-authored-by: Thamindu R <[email protected]>
@lashinijay @ThaminduR
fix formatting
9307127 
Co-authored-by: Thamindu R <[email protected]>
lashinijay and others added 2 commits January 25, 2025 01:03
@lashinijay @ThaminduR
fix license header
a00421f 
Co-authored-by: Thamindu R <[email protected]>
@lashinijay @ThaminduR
fix license header
77c9bc8 
Co-authored-by: Thamindu R <[email protected]>
@lashinijay lashinijay merged commit 194d6c1 into wso2:master Jan 24, 2025
4 checks passed
@lashinijay lashinijay mentioned this pull request Jan 27, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ThaminduR ThaminduR ThaminduR approved these changes

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

Successfully merging this pull request may close these issues.
2 participants
@lashinijay @ThaminduR