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.

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

SDK updates #56

Closed
wants to merge 4 commits into from
Closed

SDK updates #56

wants to merge 4 commits into from

Conversation

passage-beachball-bot
Copy link
Collaborator

No description provided.

Copy link

Quality Gate Passed Quality Gate passed

Issues
5 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
57.8% Duplication on New Code

See analysis details on SonarCloud

require 'openapi_client/api/magic_links_api'
require 'openapi_client/api/tokens_api'
require 'openapi_client/api/user_devices_api'
require 'openapi_client/api/users_api'
Copy link
Contributor

Choose a reason for hiding this comment

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

Failing because of the error in this file. they all need to be require_relative

Copy link
Contributor

Choose a reason for hiding this comment

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

Info here: #57


assert_equal user, user_by_identifier
end

Copy link
Contributor

Choose a reason for hiding this comment

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

Add test to check if user cannot be found.

Copy link
Contributor

@vanessa-passage vanessa-passage left a comment

Choose a reason for hiding this comment

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

Need to update the CHANGELOG and README

@vanessa-passage vanessa-passage deleted the sdk-updates-1710785404 branch November 20, 2024 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants