-
Notifications
You must be signed in to change notification settings - Fork 25
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
Possible to specify gist name? #107
Comments
Sounds like a good feature. I had this idea that maybe we could use a separator such that;
I don't know if |
Thanks for your quick fix! I am happy to take this change, but this will bring another problem, which is '-' can be used as a part of Github username. There would be a problem if a user named Suggesting to use other characters as separator, such as Reference:
|
Not sure if you like this idea |
Could this be supported via a query parameter solution?
The 'en' value would be passed as a value to a 'lang' name parameter. We know 'theme' already is a parameter, so would follow that pattern. |
That is nice as well, but I prefer to use |
query parametter seem to be fine!
all are done in getResumeGist that need to add gistname parametter. to replace resume.json filter with the value of this parametter.
they are used in two place:
in the first we can add req.query to retrive gistname parametter. there another place that reference resume.json and gist.githubusercontent.com:
in that way you can resolve mutiple version of resume and multilingale resume. |
Sounds like we should just go with the query string, seems easy to implement. |
Is your feature request related to a problem? Please describe.
I have multiple versions of resume, in different languages, but I see the registry is hardcoded the file name has to be
resume.json
, so I can only render a version of resume at one time.Describe the solution you'd like
Can we have one more parameter to specify the file name of the gist that the registry fetches from?
Describe alternatives you've considered
No alternatives
Additional context
The text was updated successfully, but these errors were encountered: