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

[AF-12] People authentication #15

Merged
merged 4 commits into from
Feb 17, 2024
Merged

[AF-12] People authentication #15

merged 4 commits into from
Feb 17, 2024

Conversation

ricardopacheco
Copy link
Owner

Issue number: resolves #12

Summary 🔴

This PR adds the user authentication feature. Additional add-ons are available in the changelog in version 0.3.1.

Proposed / Possible solution 🔴

We only added the contract and the operation, as it is an action within the user context.

How to test 👮‍♂️

Start the related services, and open the console with the command bin/console. After that:

attributes = {}

AuctionFunCore::Operations::UserContext::AuthenticationOperation.call(attributes) do |result|
  result.success { |user| puts user.to_h }
  result.failure { |failure| puts failure.to_h }
end

To simulate successful authentication, change the hash of the attributes variable so that it meets all contract requirements.

Risks / Impacts 🔴

  • None anticipated

Requirements for deployment 🔴

  • None anticipated

@ricardopacheco ricardopacheco self-assigned this Feb 17, 2024
@ricardopacheco ricardopacheco added type:feature A suggestion for general features status:in-review In Review status priority:high High priority and removed type:feature A suggestion for general features status:in-review In Review status labels Feb 17, 2024
@ricardopacheco ricardopacheco merged commit 346f64c into main Feb 17, 2024
1 check passed
@ricardopacheco ricardopacheco deleted the AF-12 branch February 17, 2024 21:05
@ricardopacheco ricardopacheco added status:done Done status and removed status:in-review In Review status labels Feb 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:high High priority status:done Done status type:feature A suggestion for general features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

People authentication
1 participant