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

As a user I want to select a template to issue a certified copy with #7813

Open
wants to merge 45 commits into
base: develop
Choose a base branch
from

Conversation

tareq89
Copy link
Collaborator

@tareq89 tareq89 commented Oct 22, 2024

@tareq89 tareq89 added this to the v1.7.0 milestone Oct 22, 2024
@tareq89 tareq89 self-assigned this Oct 22, 2024

This comment has been minimized.

@tareq89 tareq89 force-pushed the ocrvs-7584 branch 2 times, most recently from 184ba01 to f814ace Compare October 22, 2024 13:04
@ocrvs-bot
Copy link
Collaborator

Your environment is deployed to https://ocrvs-7584.opencrvs.dev

@tareq89 tareq89 force-pushed the ocrvs-7584 branch 2 times, most recently from bdc01ca to e21b0ea Compare October 29, 2024 13:32
@rikukissa
Copy link
Member

rikukissa commented Oct 31, 2024

Seems the dropdown and the radio buttons in a different order to how the design is:

image

Design
image

@tareq89
Copy link
Collaborator Author

tareq89 commented Oct 31, 2024

Seems the dropdown and the radio buttons in a different order to how the design is:

image

Design image

I did not know such design existed. Any figma link of this design was not mentioned anywhere. Anyway, I have adjusted the code as per the design.

CHANGELOG.md Outdated Show resolved Hide resolved
packages/client/src/declarations/index.ts Outdated Show resolved Hide resolved
@rikukissa rikukissa self-requested a review November 5, 2024 11:32
tareq89 and others added 18 commits November 13, 2024 14:02
* refactor: move metrics out of authenticate

* feat: initial token exchange endpoint

* chore: update comment to be more specific

* fix: issue with calling metrics environment

* chore: amend changelog

* refactor: improve clarity

* fix: unused import

* chore: add missing schema inputs

* revert: revert mosip code removal to keep PR contained

* chore: add comment about more fine-grained control

* chore: fix test on gateway

* feat: actually check if the record id matches to confirm record

* revert: the confirm registration changes - lets do inanother pr

* refactor: update error messages

* fix: make the no-op simpler

* fix: the query params not passing properly via gateway

* refactor: remove unnecessary gql inputs

* fix: update audiences to include minimum

* fix: update the todo comment
Co-authored-by: Riku Rouvila <riku.rouvila@gmail.com>
Co-authored-by: Riku Rouvila <riku.rouvila@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
In QA 🚀 Ready to deploy Deployment automation should pick this PR up and start auto-deploying it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants