Skip to content

fix(docs): add FastHTTP Postgres sample run and test instructions#766

Open
Srinu346 wants to merge 14 commits intokeploy:mainfrom
Srinu346:fix/quickstart-settings
Open

fix(docs): add FastHTTP Postgres sample run and test instructions#766
Srinu346 wants to merge 14 commits intokeploy:mainfrom
Srinu346:fix/quickstart-settings

Conversation

@Srinu346
Copy link
Contributor

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:

  • Adding the Docker option to the FastHTTP Postgres sample app.
  • Updating the relevant .md file to document both:
    • Docker-based workflow
    • Local (Linux/WSL) workflow

This PR resolves #3269

Type of change

  • Bug fix (non-breaking change which fixes an issue).
  • New feature (non-breaking change which adds functionality).
  • Breaking change (fix or feature that would cause existing functionality to not work as expected).
  • Documentation update (if none of the other choices apply).

How Has This Been Tested?

image

Checklist:

  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.

Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
@Srinu346
Copy link
Contributor Author

@Achanandhi-M sir, please take a look whenever possible.
Thank you !!!!

Copy link
Member

@Achanandhi-M Achanandhi-M left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@Srinu346
Copy link
Contributor Author

@Achanandhi-M sir i will refine it and update it.

@Srinu346
Copy link
Contributor Author

Screen.Recording.2026-01-30.001941.mp4

@Achanandhi-M sir, please take a look when possible.

@Srinu346
Copy link
Contributor Author

Hi Sir,
Currently, I’m using the previously uploaded images as-is, since they are already hosted in the AWS S3 bucket.

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.
Thank you!

Copy link
Member

@Achanandhi-M Achanandhi-M left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@Srinu346
Copy link
Contributor Author

Srinu346 commented Feb 2, 2026

@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>
@Srinu346
Copy link
Contributor Author

Srinu346 commented Feb 2, 2026

@Achanandhi-M sir here is a short video of demonstration:

Screen.Recording.2026-02-02.230840.mp4

@Srinu346
Copy link
Contributor Author

Srinu346 commented Feb 2, 2026

@Achanandhi-M Sir, i have only left out the emojis which were also being used in other setups

Copy link
Member

@Achanandhi-M Achanandhi-M left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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>
@Srinu346
Copy link
Contributor Author

Srinu346 commented Feb 3, 2026

Screenshot 2026-01-30 002806 @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.
Please let me know if any further changes are needed.
image

@Srinu346
Copy link
Contributor Author

Srinu346 commented Feb 3, 2026

@Achanandhi-M sir, here is a short video of demonstration.
Please have a look.

Recording.2026-02-03.183820.mp4

Copy link
Member

@Achanandhi-M Achanandhi-M left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Image

Current Setup in the Keploy docs:

Image

Your changes:

Image

@Achanandhi-M
Copy link
Member

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.

@Srinu346
Copy link
Contributor Author

Srinu346 commented Feb 4, 2026

Thank you sir this cleared a lot things sir.

@Srinu346
Copy link
Contributor Author

Srinu346 commented Feb 4, 2026

@Achanandhi-M sir, here is a short video.

Screen.Recording.2026-02-04.180044.mp4

I’ve reverted all Linux/WSL changes and updated only the Docker setup documentation along with the Keploy record and replay images.
For consistency with other quickstarts, I took reference from the Echo + SQL sample to follow the same structure and flow.

Thank You.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[docs]: FastHTTP + Postgres Docker Quickstart Missing

2 participants