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

feat: Dashboard verification templates editor (#2417) * Setup v #1450

Merged
merged 1 commit into from
Jul 19, 2023

Conversation

mewmba
Copy link
Contributor

@mewmba mewmba commented Jul 19, 2023

feat: Dashboard verification templates editor (#2417) * Setup verification templates routing and nav * Start building out the verification templates page * Add markup for verification template editor * Add react hook form to verification template form * Componentize sections in add verification template * Refactor Create Verification Template form * Start connecting to verification template backend * Set up backend call to create template * Reorganizing * Fixing imports * Reorganizing * Reorganizing * Renaming files * Moving stuff around * Adding provider * Adding Comps * Upgrading trinsic sdk * Moving hooks arround * reorganizing * Adding outlet for verification templates * Adding credential template id * Temp * Almost there * Adding select all buttons * Switching to checkbox * Update Trinsic SDK package version * Run make format * Fix update verification template endpoint * Remove unused console.log calls * Update server-api/Server/Services/TemplateService.cs Co-authored-by: Scott Phillips polygonguru@gmail.com --------- Co-authored-by: Hersh Patel hersh.patel@trinsic.id Co-authored-by: Scott Phillips polygonguru@gmail.com


PR-Codex overview

Focus of the PR:

This PR focuses on updating the VerificationTemplateFieldPatch message in the templates.proto file.

Detailed summary:

  • Removed the usage_policy field from the VerificationTemplateFieldPatch message.
  • Added the field_share_type field of type VerificationShareType to the VerificationTemplateFieldPatch message.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@mewmba mewmba requested a review from lucasamonrc July 19, 2023 20:14
@lucasamonrc lucasamonrc added this pull request to the merge queue Jul 19, 2023
Merged via the queue into main with commit cdc46a3 Jul 19, 2023
12 of 30 checks passed
@lucasamonrc lucasamonrc deleted the mewmba/main-68cbe64a-generated-protos branch July 19, 2023 20:15
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.

None yet

2 participants