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

Release 4.4.0 #245

Merged
merged 45 commits into from
Oct 30, 2023
Merged

Release 4.4.0 #245

merged 45 commits into from
Oct 30, 2023

Conversation

mpgxvii
Copy link
Member

@mpgxvii mpgxvii commented Oct 25, 2023 •
edited
Loading

blootsvoets and others added 30 commits March 23, 2023 13:19
@blootsvoets
Bump dev versions
5807030
@blootsvoets
Readability fixes
51b1c91
@mpgxvii
Store auth params in authorization page as well
a43e0cd
@mpgxvii
Merge pull request #235 from RADAR-base/fix/garmin-auth-url
13fb5a4 
Store auth params in authorization page
@blootsvoets
Update to latest radar-auth
0b05de8
@blootsvoets
Code cleanup
e3c468d
@blootsvoets
Merge branch 'master' into dev
082bc0e
@blootsvoets
Use radar-commons plugin
7c1e40c
@blootsvoets
Bump dockerfile versions
a18a121
@blootsvoets
Bump dependencies and optimize imports
c074635
@blootsvoets
Small build cleanup
92ee7e0
@blootsvoets
Merge branch 'dev' into update-radar-auth-2
28a6974
@Hsankesara
Adding OuraAuthSerive and running it
f76adb1
@Hsankesara
added Oura OAuth 2
34f323a
@Hsankesara
merged dev and updated code
a1c9c7c
@Hsankesara
updated Resources
bd7ebb1
@Hsankesara
fixed lint issues and solve permission error
d70867a
@Hsankesara
imports are in lexicographic order now
10e041b
@Hsankesara
minor changes from the PR review
43f8519
@Hsankesara
updated authorizer.yml
0d66fd9
@Hsankesara
added ring_configuration scope in Oura
b35367a
@Hsankesara
Merge pull request #241 from RADAR-base/Oura-oauth2
3bfefd2 
Oura Oauth2 Integration
@Hsankesara
Added code to get external id during auth Oura
3e56a2e
@Hsankesara
added exception handling
f53462e
@Hsankesara
fixed kotlin lint issue
b09b453
@Hsankesara
fixed few other klint issue
a2f316a
@Hsankesara
simplify oura requestAccessToken function
c28db9f
@blootsvoets
Bumped dependencies
c9b0c3a
@Hsankesara
updated Oura getExternalId fun
9357146
@Hsankesara
solved lint issues I think
99a0431
blootsvoets
blootsvoets previously approved these changes Oct 25, 2023
View reviewed changes
Copy link
Contributor

@blootsvoets blootsvoets left a comment

Choose a reason for hiding this comment

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

LGTM. @peyman-mohtashami @Bdegraaf1234 would you take a look as well?

@mpgxvii mpgxvii marked this pull request as ready for review October 25, 2023 13:13
yatharthranjan
yatharthranjan approved these changes Oct 30, 2023
View reviewed changes
Copy link
Member

@yatharthranjan yatharthranjan left a comment

Choose a reason for hiding this comment

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

LGTM, will test further on stage env.

@mpgxvii mpgxvii merged commit 6867f5d into master Oct 30, 2023
5 checks passed
@mpgxvii mpgxvii deleted the release-4.4.0 branch January 25, 2024 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yatharthranjan yatharthranjan yatharthranjan approved these changes

@blootsvoets blootsvoets blootsvoets left review comments

@peyman-mohtashami peyman-mohtashami Awaiting requested review from peyman-mohtashami

@Hsankesara Hsankesara Awaiting requested review from Hsankesara

@Bdegraaf1234 Bdegraaf1234 Awaiting requested review from Bdegraaf1234

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

Successfully merging this pull request may close these issues.
4 participants
@mpgxvii @blootsvoets @yatharthranjan @Hsankesara