fix(docs): add FastHTTP Postgres sample run and test instructions#766
fix(docs): add FastHTTP Postgres sample run and test instructions#766Srinu346 wants to merge 14 commits intokeploy:mainfrom
Conversation
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
|
@Achanandhi-M sir, please take a look whenever possible. |
Achanandhi-M
left a comment
There was a problem hiding this comment.
Hey @Srinu346, please take a look at the other quickstarts to understand how they’re structured. What you’ve added currently feels more like a README, whereas the docs follow a different format. Take your time to review how we’ve added other quickstarts and align this with our existing guides.
Also, please include a video demonstrating your changes and add relevant images as well. Make sure the images have a white background.
|
@Achanandhi-M sir i will refine it and update it. |
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
Screen.Recording.2026-01-30.001941.mp4@Achanandhi-M sir, please take a look when possible. |
|
Hi Sir, For reference, these are the images being used:
Please let me know if this is okay, or if you would prefer me to update/replace them with new screenshots. |
Achanandhi-M
left a comment
There was a problem hiding this comment.
Hey @Srinu346, please avoid removing the existing image from the file—don’t change anything there and use it as is. You can include additional images in the directory if needed; that’s not an issue.
Also, please include a short video showcasing your changes. One more request: kindly avoid using AI-generated text, as it appears that may have been used, and please remove emojis as well. Thanks for understanding.
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
…/docs into fix/quickstart-settings
|
@Achanandhi-M sir, Regarding emojis, I have removed most of the unnecessary ones and retained only those that are already used in other quickstart samples for consistency. I have also revised the content to remove any AI-generated text and updated it accordingly. Please let me know if any further changes are required. |
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
|
@Achanandhi-M sir here is a short video of demonstration: Screen.Recording.2026-02-02.230840.mp4 |
|
@Achanandhi-M Sir, i have only left out the emojis which were also being used in other setups |
Achanandhi-M
left a comment
There was a problem hiding this comment.
The changes look good, @Srinu346 . However, please don’t modify anything in the existing setup—only include the required updates. I noticed that some changes were made to the Docker Compose method; kindly avoid that. Also, I don’t see the images for the Keploy test that were mentioned in the previous review—please add those as well.
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
@Achanandhi-M sir,
I have added the previously removed part back into the Docker Compose section. I’ve also included the above Keploy test image at the appropriate places both in the Docker Compose section and the WSL/Linux section. I haven’t removed or modified any of the existing images; only added the new ones where relevant. I’ve tried to limit the changes strictly to the sections where updates were required. |
|
@Achanandhi-M sir, here is a short video of demonstration. Recording.2026-02-03.183820.mp4 |
There was a problem hiding this comment.
Hey @Srinu346, please follow my previous review. I had mentioned not to make any changes to the existing setup, but I still see a few changes there. If there was a specific reason for modifying the existing setup, could you please explain what led to those changes? I’ve attached screenshots for your reference.
Also, for the Linux setup, please follow the approach used in the other Linux setups so we can improve it in a similar way. Additionally, please update the image for the Docker Compose setup—the current image shows the Linux setup. As mentioned in the previous feedback, please add a separate image specifically for the Docker Compose setup. Thanks for your understanding.
Current Setup in the Keploy docs:
Your changes:
|
Hey @Srinu346, the approach is quite simple. We only want to include the Docker setup, so please add documentation specifically for the Docker setup. If you have any doubts, you can refer to the existing docs. Also, please include images for the Keploy record and replay steps. Don’t worry about the S3 part—I’ll handle the upload from my side once you’ve created the images. Lastly, please don’t change any existing steps in the Linux setup. That should be all. |
|
Thank you sir this cleared a lot things sir. |
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
…/docs into fix/quickstart-settings
|
@Achanandhi-M sir, here is a short video. Screen.Recording.2026-02-04.180044.mp4I’ve reverted all Linux/WSL changes and updated only the Docker setup documentation along with the Keploy record and replay images. Thank You. |


What has changed?
This PR updates the FastHTTP + Postgres sample documentation to expose and document the existing Docker-based setup that was already implemented in the backend.
The changes include:
.mdfile to document both:This PR resolves #3269
Type of change
How Has This Been Tested?
Checklist: