-
Notifications
You must be signed in to change notification settings - Fork 0
Chore simplify codebase #1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Removed redundant comments - Clarified client logic
- Removed unused settings logic - Streamlined status updates - Removed demo data migration
- Reduced destinations to one - Removed unused l10n - Cleaned up comments
- Updated package imports - Removed unused repositories - Simplified app initialization
- Removed unused event - Removed handler function
- Removed unused dependencies - Streamlined redirect logic - Added dashboard route
- Created basic dashboard UI - Added welcome text
- Added email auth translations - Supports English and Arabic
- Removes splash on web after app loads - Improves user experience
- Added email sign in strings - Added code verification strings
- Removed isLinkingContext prop - Simplified navigation logic - Removed conditional close button
- Added labels for dashboard - Added labels for content management - Added labels for headlines - Added labels for categories - Added labels for app configuration
- Added l10n keys for nav labels - Used l10n for navigation labels
- Added content management routes - Added app configuration route - Added settings route
- Created basic page structure - Added placeholder UI
- Created basic UI - Added placeholder text
- Added settings page - Placeholder implementation
- Added Headlines tab - Added Categories tab - Added Sources tab - Implemented tab controller
- Removed unnecessary Scaffold - Simplified page structure
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Status
READY/IN DEVELOPMENT/HOLD
Description
Type of Change