-
Notifications
You must be signed in to change notification settings - Fork 2
Suggested changes from proofreading review (fiverr.com/textandtables/) #122
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
base: master
Are you sure you want to change the base?
Conversation
Includes comments with syntax: (TTNote: ... )
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
Includes comments with syntax (TTNote: ... ) that can be removed before publishing.
augustak
left a comment
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.
Hi @textandtables ,
Thank you for many great changes! However, it looks like you have removed the table of contents as well as some illustrations. Can you have a look? Also, anything in the 'reference' section is autogenerated into this repository and shouldn't be edited in this repository directly but rather in their respective origin repository. Can you remove those changes from this PR and optionally add them back to their respective repositories?
https://github.com/LucidtechAI/las-cli
https://github.com/LucidtechAI/las-sdk-python
https://github.com/LucidtechAI/las-sdk-net
https://github.com/LucidtechAI/las-sdk-java
https://github.com/LucidtechAI/las-sdk-js
I will start reviewing once those things have been fixed :-)
|
Hi @augustak - Thank you for your feedback on my review. I apologize for any difficulties that I may have caused. Regarding the table of contents, I did not see a section with this name while I was reviewing the changes, so if you could point me to the correct page where this is located or the change number that I have added, I will be happy to address it. As fyi, I have made all the changes via GitHub not GitBook, per my discussions with @staalezh. I’m not sure if that’s relevant to how the table of contents are displayed. Regarding the reference section, I’m sorry that I was not aware that these should not be changed. I am happy to provide my suggestions in an alternate form (if you could be so kind as to provide me instructions), and then I will remove the changes from GitHub. For clarity, my understanding is that this encompasses my 12 changes under las-docs /reference with filenames latest.md and README.md, with the initial ID’s below, that I made on Jun 22, 2021: Thanks in advance. I look forward to your feedback. |
Hi Chris, It looks like "Gitbook" has made the changes that removed table of contents and illustrations. Can you try reverting the following commits: I deployed your fork here so that you can view how your changes will be reflected on the website. You are correct that the 'reference' section I'm referring to is located in the '/reference' folder (and subfolders) and that you can revert those changes in this repository, but add them in their respective location in the SDK/CLI repositories that I have linked above. Sorry for the inconvenience, Best, |
|
Hi @augustak and thanks for your feedback. I was off for a US holiday and just returning to work today. Please give me 1-2 days to review and follow your suggestions. Thanks. |
|
Hi @augustak, Thank you again for taking a look at this and for your suggestions to resolve the issues. I see the first commit (5532f08) covers the missing images, a README file (for which I only made a minor suggestion) and a SUMMARY file (which I did not intentionally update). But as I am not a GitHub expert, I would appreciate instructions on how to revert my commits - I don’t see a way to do this. Regarding the reference section, please clarify where you would like my suggested changes moved to. I’m happy also to revert those commits to avoid any conflicts for you. Thanks in advance, |
Test Change to Transitions & Workflows (added line 6)