This repository has been archived by the owner on Apr 17, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 73
feat(templates): adding external templates #2215
Closed
Closed
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit
Hold shift + click to select a range
949d11f
feat(templates): ability to add external templates
RinkiyaKeDad 4ca9416
feat(templates): split templates into categories
RinkiyaKeDad a76f94f
feat(template): minor fixes
RinkiyaKeDad f12aad0
feat(templates): changed display style
RinkiyaKeDad a2b576f
feat(templates): linting fix
RinkiyaKeDad 8b103c1
feat(templates): linting fix 2
RinkiyaKeDad f6a8399
feat(templates): cleared json file
RinkiyaKeDad c2d84c6
feat(templates): removed external flag
RinkiyaKeDad b8d9c69
feat(templates): fixed build error due to empty json file
RinkiyaKeDad 5dabe76
feat(templates): fixed build errors
RinkiyaKeDad 1910103
feat(templates): fixed another import
RinkiyaKeDad d1d9974
docs(templates): added draft doc
RinkiyaKeDad 1fbba2c
feat(templates): switched from json file to ts
RinkiyaKeDad File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -14,6 +14,7 @@ | |
"branch": "templates-1.0.0", | ||
"path": "/templates/ts-apollo-postgres-backend" | ||
} | ||
] | ||
], | ||
"external": true | ||
} | ||
] |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -32,7 +32,7 @@ async function chooseTemplate(filter: string = ''): Promise<Template> { | |
{ | ||
type:'list', | ||
name:'typeOfTemplates', | ||
message:'Choose between community or officail Graphback templates', | ||
message:'Choose between community or official Graphback templates', | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Would it be possible to print them separately? Too much ask? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Separately as in no choosing and just all of them listed with some gap (or a line saying that 'following are the community templates') in between? |
||
choices:['community','official'] | ||
} | ||
]); | ||
|
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Idea is that this file will be for external ones so do we need flag
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.
We would need the flag somewhere. If not here then I could have it for the internal templates, something like
internal: true
?