Rename demo-app to Remote Notify Demo App #8
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
…t history - Use git mv to preserve file history when moving Kotlin files - Change package name from org.nella.fcmapp to org.nella.RN.demo (capital RN) - Update applicationId and namespace in build.gradle to new package name - Rename FCMService to RnDemoService class and update all references - Update AndroidManifest.xml with new service name and theme references - Change app name from 'FCM App' to 'Remote Notify Demo App' in strings.xml - Update theme name from Theme.FCMApp to Theme.RemoteNotifyDemoApp - Replace user-facing 'FCM' references with 'notification' terminology - Change default backend URL to https://demo.rn.nella.org - Update README.md with new app name, package references, and default URL - Maintain Firebase-specific technical terms where appropriate - Update SharedPreferences name from FCMAppSettings to RnDemoSettings - Change notification channel ID from fcm_default_channel to rn_demo_channel - Update all UI hints and examples to use new demo.rn.nella.org domain - Preserve all Firebase Cloud Messaging functionality and dependencies Co-Authored-By: sketch <hello@sketch.dev> Change-ID: s2f8322e1907a4366k
- Change package from org.nella.RN.demo to org.nella.rn.demo (lowercase rn) - Use git mv to rename RN directory to rn preserving file history - Update package declarations in all Kotlin files - Update namespace and applicationId in build.gradle - Update README.md with correct package name - Resolve Firebase google-services.json package name mismatch Co-Authored-By: sketch <hello@sketch.dev> Change-ID: s5a58cb7992b18edek
- Add cache: false to setup-go@v4 actions to disable automatic caching - Resolve 'Dependencies file is not found' error in GitHub Actions - Go modules are in subdirectories (app-backend/, notification-backend/) - Automatic cache detection expects go.mod in repository root - Disabling cache avoids dependency file location issues while maintaining functionality Co-Authored-By: sketch <hello@sketch.dev> Change-ID: sc6ed588fe30ae39ak
- Add cache-dependency-path to setup-go@v4 actions pointing to actual Go dependency files - Configure caching for app-backend/go.mod and notification-backend/go.sum - Enable Go module caching for both test-go-servers and go-lint jobs - Resolve 'Dependencies file is not found' error by specifying correct paths - Improve CI performance with proper dependency caching instead of disabling it - Cache will now work correctly with Go modules in subdirectories Co-Authored-By: sketch <hello@sketch.dev> Change-ID: s3506506da8c0c492k
- Configure cache-dependency-path to point only to notification-backend/go.sum - app-backend has no external dependencies so no go.sum file is generated - GitHub Actions Go caching specifically requires go.sum files for proper operation - Run go mod tidy on notification-backend to ensure dependencies are current - Simplify cache configuration to only reference files that actually exist - This resolves caching issues while maintaining optimal CI performance Co-Authored-By: sketch <hello@sketch.dev> Change-ID: sc9d33bd20f67f9c0k
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.
Change-ID: s2f8322e1907a4366k