Skip to content

Conversation

AayushTyagi1
Copy link
Contributor

No description provided.

@AayushTyagi1 AayushTyagi1 requested a review from a team as a code owner March 11, 2025 07:08
@AayushTyagi1 AayushTyagi1 self-assigned this Mar 11, 2025
@AayushTyagi1 AayushTyagi1 requested a review from rajagp March 11, 2025 07:08
Copy link

@deniswsrosa deniswsrosa left a comment

Choose a reason for hiding this comment

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

Tutorial needs considerable rework

---

## Introduction

Choose a reason for hiding this comment

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

Needs considerable rework. This is not a tutorial at all, you are just explaining how to run it with next to no explanation of what it is. Anyone landing on this article would be "Completely Lost" on what it is about.

What is edge server? What are you trying to teach in this tutorial? why do I need sync gateway? Why should I care about this in the first place? Is there any interesting piece of code that you should mention to the user? there are so much to unpack here it is even hard to mention what is missing.

Your tutorial needs to sell/get the user interested in the feature. Right now, we just show how to run the app with virtually zero context.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@rajagp, it looks like this tutorial needs some rework. Since you wrote it, could you please make the necessary changes?
If you want me to take it, I will work on it tomorrow.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@rajagp

I’ve revised to the tutorial based on the above feedback. Could you please check and make the necessary changes from your side? I’ll be OOO next week from tomorrow, feel free to merge the PR once everything looks good to you and Denis.

Copy link

@deniswsrosa deniswsrosa left a comment

Choose a reason for hiding this comment

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

Looks much better Kush! Thank you.

@AayushTyagi1 AayushTyagi1 merged commit 31a315b into main Apr 3, 2025
4 checks passed
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.

2 participants