Skip to content

Improved README with architecture, better navigation and cleaner structure!#197

Open
niyatisharma100307-crypto wants to merge 2 commits into
rdodiya:gssoc_developfrom
niyatisharma100307-crypto:gssoc_develop
Open

Improved README with architecture, better navigation and cleaner structure!#197
niyatisharma100307-crypto wants to merge 2 commits into
rdodiya:gssoc_developfrom
niyatisharma100307-crypto:gssoc_develop

Conversation

@niyatisharma100307-crypto
Copy link
Copy Markdown
Contributor

What I changed in this PR!!

🗑️ Removed

  • Removed broken screenshot placeholders (Show Image boxes)
    ➕ Added
  • Added architecture diagram showing how frontend, backend and database connect
  • Added Prerequisites table with download links for all tools
  • Added PR checklist for contributors
  • Added code style guidelines for Java and React
  • Added branch naming conventions (feature/, fix/, docs/, etc.)
  • Added "stay up to date with upstream" step in contribution guide
  • Added color-coded difficulty table for open issues

🔧 Fixed

  • Fixed broken TOC links for Tech Stack and Architecture sections
    (caused by special emoji characters that GitHub couldn't read)
  • Replaced 🏗️ with 🔧 and 🏛️ with 📐 to fix anchor links

♻️ Improved

  • Rewrote Quick Start with clear numbered steps
  • Converted Tech Stack from bullet points to tables (shows version + role)
  • Split Features into three clear sections: Available / In Progress / Roadmap
  • Converted Troubleshooting into collapsible dropdowns (cleaner look)
  • Improved Contact section into a clean table layout
  • Added 2 extra badges (PostgreSQL, Gradle) to the header

📄 Type of change

  • Documentation only (no code was changed)

closes #137

@niyatisharma100307-crypto
Copy link
Copy Markdown
Contributor Author

Any update?!

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR focuses on improving the main project README to make onboarding, navigation, and contribution guidance clearer, aligning with issue #137’s request for better quick start, architecture visibility, and contributor workflow documentation.

Changes:

  • Reworked README structure and navigation (updated header, TOC, reorganized sections).
  • Added onboarding enhancements (prerequisites table, clearer quick start steps, troubleshooting details).
  • Expanded contributor guidance (branch naming, commit format, PR checklist, code style notes) and added an architecture diagram.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ReadMe.md
Comment on lines +141 to 145
psql --version # expect 14+
git --version
=======
- Node.js 18.0 or higher
- npm 9.0 or higher (comes with Node.js)
Comment thread ReadMe.md
Comment on lines +291 to 297
```env
# Spring Boot context path — no trailing slash
VITE_API_BASE_URL=http://localhost:8181/restroly
=======
#### 2. Environment configuration - Google OAuth

Create a `.env` file in `RestroHub-FrontEnd/` (see `.env.example`):
Comment thread ReadMe.md
Comment on lines +309 to 313

# Optional
VITE_NODE_ENV=development
VITE_ENABLE_ANALYTICS=false
```
Comment thread ReadMe.md
├── CONTRIBUTING.md
├── LICENSE
└── CONTRIBUTING.md
└── README.md
Comment thread ReadMe.md
<summary><strong>Gradle wrapper JAR missing</strong></summary>

```bash
gradle wrapper --gradle-version 8.7
Comment thread ReadMe.md
Comment on lines +437 to +440
| `GET` | `/api/v1/menus` | List all menus |
| `POST` | `/api/v1/menus` | Create a menu |
| `PUT` | `/api/v1/menus/{id}` | Update a menu |
| `DELETE` | `/api/v1/menus/{id}` | Delete a menu |
Comment thread ReadMe.md
Comment on lines 442 to +461
#### Categories
- `GET /api/v1/categories` - Get all categories
- `POST /api/v1/categories` - Create category
| Method | Endpoint | Description |
|--------|----------|-------------|
| `GET` | `/api/v1/categories` | List all categories |
| `POST` | `/api/v1/categories` | Create a category |

#### Food Items
- `GET /api/v1/foods` - Get all food items
- `POST /api/v1/foods` - Add food item
- `PUT /api/v1/foods/{id}` - Update food item
- `DELETE /api/v1/foods/{id}` - Delete food item
| Method | Endpoint | Description |
|--------|----------|-------------|
| `GET` | `/api/v1/foods` | List all food items |
| `POST` | `/api/v1/foods` | Add a food item |
| `PUT` | `/api/v1/foods/{id}` | Update a food item |
| `DELETE` | `/api/v1/foods/{id}` | Delete a food item |

#### Orders
- `GET /api/v1/orders` - Get all orders
- `POST /api/v1/orders` - Place new order
- `GET /api/v1/orders/{id}` - Get order details
| Method | Endpoint | Description |
|--------|----------|-------------|
| `GET` | `/api/v1/orders` | List all orders |
| `POST` | `/api/v1/orders` | Place a new order |
| `GET` | `/api/v1/orders/{id}` | Get order details |
@rdodiya
Copy link
Copy Markdown
Owner

rdodiya commented Jun 7, 2026

Hi @niyatisharma100307-crypto ,
Please review the above points and update the ReadMe file.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve README for Better Onboarding, Contribution Flow & Feature Visibility

3 participants