[INJICERT-657] Added DB upgrade scripts for release #146
Conversation
Signed-off-by: Piyush7034 <[email protected]>
Signed-off-by: Piyush7034 <[email protected]>
| COMMENT ON COLUMN svg_template.context IS 'VC Context: Context URL list items separated by comma(,)'; | ||
| COMMENT ON COLUMN svg_template.credential_type IS 'Credential Type: Credential type list items separated by comma(,)'; | ||
| COMMENT ON COLUMN svg_template.template IS 'Template Content: Velocity Template to generate the VC'; | ||
| COMMENT ON COLUMN svg_template.cr_dtimes IS 'Date when the template was inserted in table.'; | ||
| COMMENT ON COLUMN svg_template.upd_dtimes IS 'Date when the template was last updated in table.'; |
There was a problem hiding this comment.
Fixed template name
Piyush7034
force-pushed
the
injicert-499
branch
from
f3bc97e to
e7ae5df
Compare
| @@ -0,0 +1,33 @@ | |||
| \c inji_certify | |||
|
|
|||
| CREATE TABLE svg_template ( | |||
There was a problem hiding this comment.
can we rename this to rendering template?
the type of image used here is svg, it can be changed as well right?
There was a problem hiding this comment.
with additional field designating the type of template
There was a problem hiding this comment.
These changes have to be merged in release branch also and there are templates and repositories associated with this table. Can these changes be taken up for the next release? @vharsh
There was a problem hiding this comment.
We're a bit late in designing this template as this change has also gone to released and collab, hence I suggest this to be taken up in next release if possible.
README.md
Outdated
| 1. Create the tables with all the fields that should be returned by the Postgres Data Provider Plugin within the certify postgres database. | ||
| - Refer the following query for insertion in DB: | ||
| ``` | ||
| CREATE TABLE certify.<table_name> ( |
There was a problem hiding this comment.
are we finalizing this design for certify pg plugin?
data present within certify DB?
There was a problem hiding this comment.
There was a problem hiding this comment.
This doc refers to the configuration setup for postgres plugin as requested by Likhitha. So it was suggested to add this in the certify docs for reference. This should be updated based on future changes.
There was a problem hiding this comment.
No, but this is being given as a handover to the QA team and a minimal documentation is required for the same, can we drop this plugin from the scope?
There was a problem hiding this comment.
This document is now moved to digital-credential-plugins repo under postgres-dataprovider-plugin project as suggested by @vishwa-vyom
Signed-off-by: Piyush7034 <[email protected]>
Piyush7034
force-pushed
the
injicert-499
branch
from
e7ae5df to
30efe6c
Compare
Signed-off-by: Piyush7034 <[email protected]>
Piyush7034
changed the title
Signed-off-by: Piyush7034 <[email protected]>
No description provided.