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

Feat(auth): Remove auth provider entity #6314

Merged
merged 105 commits into from
Feb 6, 2024
Merged

Feat(auth): Remove auth provider entity #6314

merged 105 commits into from
Feb 6, 2024

Conversation

pKorsholm
Copy link
Contributor

What

  • remove auth provider entity

Why

  • The auth provider entity was not really used anywhere

How

  • Keeping loader behavior as is but removing the

@pKorsholm
initial implementation
dacdb27
@pKorsholm
add test for invalid scope
cd38e4c
@pKorsholm
get config from scope not db
d3fd771
@pKorsholm
assign config from scope
512f5df
@pKorsholm
fix package.json
908e5e6
@pKorsholm
optional providers
4bd6491
@pKorsholm
make providers options
9869c71
@pKorsholm
rename auth to authentication
36d51ef
@pKorsholm
more renaming
7553ea0
@pKorsholm
update package name
ecd2250
@pKorsholm
add create-customer and customers/me routes
333a268
@pKorsholm
add changeset
036e577
@pKorsholm
rm auth endpoints from branch
9d49a5d
@pKorsholm
cleanup for pr
99c8e2d
@pKorsholm
add fields to customer creation endpoint
94cf842
@pKorsholm
add fields to customer creation endpoint
e6baee6
@pKorsholm
delete file
76ca08e
@pKorsholm
update query-config
0418435
@pKorsholm
rename steps
fe2c5c3
@pKorsholm
add create-customer and customers/me routes
d8e5415
@pKorsholm
add changeset
9c6a487
@pKorsholm
rm auth endpoints from branch
5cc7742
@pKorsholm
cleanup for pr
44dbffd
@pKorsholm
add fields to customer creation endpoint
d33f05d
@pKorsholm
add fields to customer creation endpoint
e4b0808
@pKorsholm
delete file
560162a
@pKorsholm
update query-config
2742d5d
@pKorsholm
rename steps
b92d8a0
@pKorsholm
add auth folders
e6a4d59
@pKorsholm
add middlewares
58144bb
srindom
srindom approved these changes Feb 5, 2024
View reviewed changes
Copy link
Collaborator

@srindom srindom left a comment

Choose a reason for hiding this comment

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

LGTM

@srindom srindom mentioned this pull request Feb 5, 2024
Merged
@pKorsholm pKorsholm changed the base branch from feat/token-auth to develop February 5, 2024 08:53
@kodiakhq kodiakhq bot merged commit 882aa54 into develop Feb 6, 2024
17 checks passed
@kodiakhq kodiakhq bot deleted the feat/remove-auth-provider branch February 6, 2024 07:54
This was referenced Feb 6, 2024
Merged
Closed
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adrien2p adrien2p adrien2p left review comments

@srindom srindom srindom approved these changes

Assignees
No one assigned
Labels
automerge team: medusa
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pKorsholm @srindom @adrien2p