-
Notifications
You must be signed in to change notification settings - Fork 10
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
some improvements to docs site #11
Comments
will do by Thursday or Saturday. |
please try to deliver at least by my Thursday morning (your Thursday night) @tfsojon |
@tfsojon |
updated doc - #18 What you have asked in the last video that is possible but not in this codebase also need time to achieve that. |
Which changes does PR 18 have? In the above two videos, which video did you finish in PR18? @tfsojon all the new requirements are coming now (and going to come in future too) is because, as we are using Hugo now we are discovering more needs and as we are using your template now to add our content we are discovering more needs. Plus based on marketing team suggestions we are getting new ideas and requirements. Hard to guess and give you a structure already. The template that I bought from your company is good, but just needs few enhancements according to our needs. |
@tfsojon whatever the changes you made in PR18, please cancel and re-submit it. First, do a git pull of chandu branch and then copy/make your changes on the latest chandu branch and submit PR please. When I checked your PR18, all the docs I added to my chandu branch were deleted in yours. Now, if I merge your branch into mine, I will lose everything. So, please update your local chandu branch, add the new changes to it and then submit PR. |
I think I got what you did in the content folder (after I did left-hand vs right-hand view). I am trying it out in my local. I will let you know if I still need help, @tfsojon with merging PR 18. |
@tfsojon |
@tfsojon - before fixing the mentioned issue, please do a git pull to get the chandu branch changes into your local. |
Please check the new PR |
@tfsojon
please finish these quickly
https://www.loom.com/share/f80ee6f9088c4aa9835b41ae4948074e
The text was updated successfully, but these errors were encountered: