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

0.3.0 release (develop -> master) #22

Merged
merged 45 commits into from
Nov 22, 2024
Merged

0.3.0 release (develop -> master) #22

merged 45 commits into from
Nov 22, 2024

Conversation

Xiangs18
Copy link
Collaborator

No description provided.

MrCreosote and others added 30 commits April 24, 2024 11:45
@MrCreosote
Add GHA and simplify test setup
4a5e1e3
@MrCreosote
Fix Docker build
9c4a73d
@MrCreosote
Merge pull request #12 from kbase/dev-gradle
98cfe2f 
Add GHA and simplify test setup
@MrCreosote
Switch to gradle
6b3b9f3
@MrCreosote
Move source to standard Gradle locations
bcc6352
@MrCreosote
Remove lookup_globus_user method
5a91b7d 
Unused in KBase as per a github code search and Erik P.

auth2 service has an endpoint with equivalent functionality
@MrCreosote
Update dockerfile w/ gradle build & add compose file
8986759 
Remove old build cruft
@MrCreosote
Switch to jitpacked jars
223a916
@MrCreosote
Merge pull request #13 from kbase/dev-gradle
8d04983 
Switch to gradle
@MrCreosote
Merge pull request #14 from kbase/dev-gradle
a4225d8 
Move source to standard Gradle locations
@MrCreosote
Merge pull request #15 from kbase/dev-gradle
d41a348 
Remove lookup_globus_user method
@MrCreosote
Merge pull request #16 from kbase/dev-gradle
5842f5b 
Update dockerfile w/ gradle build & add compose file
@MrCreosote
Merge pull request #17 from kbase/dev-gradle
748832d 
Switch to jitpacked jars
@Xiangs18
mongo7 upgrade
27c33a5
@Xiangs18
fix JsonParseException: Unrecognized token
03d4512
@Xiangs18
switch to BsonDocument.parse
4b2ae15
@Xiangs18
check jsonString
ecea370
@Xiangs18
change toString() to toJson()
fd3d525
@Xiangs18
fix updateProfile logic
bd65c01
@Xiangs18
fix inline code
f91177b
@Xiangs18
add test coverage for getProfile function
629a71c
@Xiangs18
cover filterUsers missing lines
a551db4
@Xiangs18
Test the updateProfile function on a non-existent user
1457a88
@Xiangs18
set admin update nonExistProfile
a3f1109
@Xiangs18
simplify if else statement
b90d710
@Xiangs18
update release notes
48e5aa0
@Xiangs18
refactor code and remove old comments
053b474
@Xiangs18
Update Specific Fields in profile
bc602a2
@Xiangs18
fix bug
973557d
@Xiangs18
remove print statements
4fa6c34
Xiangs18 added 15 commits October 4, 2024 13:40
@Xiangs18 Xiangs18 requested a review from MrCreosote November 20, 2024 23:51
MrCreosote
MrCreosote approved these changes Nov 21, 2024
View reviewed changes
Copy link
Member

@MrCreosote MrCreosote left a comment

Choose a reason for hiding this comment

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

Obviously not going to go through this line by line as it's all been reviewed already, but skimming the file changes and doing some spot checks looks good

@Xiangs18 Xiangs18 merged commit a897119 into master Nov 22, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MrCreosote MrCreosote MrCreosote 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
@Xiangs18 @MrCreosote