-
Notifications
You must be signed in to change notification settings - Fork 26
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
Feature/eid login #656
Feature/eid login #656
Conversation
* master: (73 commits) Fix journey link Fix typos Fix typo (#516) Do not fill municipality_identifier if not SR adress Make different wording for cb and person stakeholders Parse stakeholder prefixes from RPO formatted name if missing in structured data Fix incorrect municipality wording Search municipality options by prefix if no exact match Add div id for or-sr-app Update date of birth validations Add better date of birth validations Do not support cases with stakeholders without deposit entries, Make deposit float if needed, Evaluate other_identifier length Fix person stakeholder names Add props to plausible event, move script to assets ND-150 Pridanie kategórií do modelu (#490) Scope down Journeys and Pages which are published to sitemap.xml (#507) Add script to log submission events to Plausible Update municipality code list Add social share image Support cases with missing deposit entries & Do not fill identifiers in 'Vymaz' part, fill original municipality ... # Conflicts: # db/structure.sql
Co-authored-by: Lucia Janíková <[email protected]>
Co-authored-by: Lucia Janíková <[email protected]>
…or safety reasons
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nejake mini zmeny a mozeme. Test na toto je asi problem spravit ci?
) | ||
end | ||
|
||
def self.assertion(eid_token, client: Faraday, url: "#{ENV.fetch('AUTH_EID_BASE_URL')}/api/upvs/assertion?token=#{eid_token&.api_token}") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@celuchmarek @luciajanikova toto potrebujeme pridat na prod aj staging.
@luciajanikova @celuchmarek toto je ready na merge, ale potrebujeme nastavit ENVs. |
@jsuchal nahodil som env |
No description provided.