Skip to content

Duplicate route defination#72

Merged
RUKAYAT-CODER merged 1 commit intorinafcode:mainfrom
Nabeelahh:feat/Added-duplicate-route-defination
Apr 23, 2026
Merged

Duplicate route defination#72
RUKAYAT-CODER merged 1 commit intorinafcode:mainfrom
Nabeelahh:feat/Added-duplicate-route-defination

Conversation

@Nabeelahh
Copy link
Copy Markdown
Contributor

closes #47

Overview
This PR resolves the code duplication issue where screen components existed in both src/screens/ and app/ directories, causing maintenance overhead and routing confusion.

🎯 Background

Screens were previously defined in two locations:

  • src/screens/*.tsx - Legacy screen components
  • app/(tabs)/ and app/*.tsx - Expo Router screens

This duplication required developers to maintain two versions of the same screens, leading to:

  • Inconsistent UI between routes
  • Increased maintenance burden
  • Confusion about which file to modify

✅ Changes Made

  • Removed all duplicate screen definitions from src/screens/ directory
  • Migrated all unique screen components to app/ directory following Expo Router conventions
  • Updated import paths to reflect new screen locations
  • Verified routing works correctly with consolidated structure

📁 Impacted Files

Removed:

  • src/screens/*.tsx (all screen components)

Modified:

  • app/(tabs)/ - Updated to use migrated screen components
  • app/*.tsx - Consolidated route files

@RUKAYAT-CODER RUKAYAT-CODER merged commit 0a14277 into rinafcode:main Apr 23, 2026
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.

Duplicate Route Definitions

2 participants