-
-
Notifications
You must be signed in to change notification settings - Fork 45
Feature/connect #3088
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
Feature/connect #3088
Conversation
Removed adding Go to connect jobs in dropdown in login screen
Drawing a red circle on the icon when there are unread messages.
…ssage is received for that channel and the key hasn't been retrieved yet.
…e string for received messages
… for light and one for dark mode.
Disable menu items while downloading the applications
…ing encryption key.
Messaging icon improvements
Messaging push notification improvements
Delete SSO token storage when token auth fails
Automatic encryption key retrieval for messaging channels
Preview of message in messages channel changed
…nto dv/connect_initial
Brings back missing analytics events And Token Exceptions Handling
Remove connect enable setting
Correct method name
|
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughThis change set introduces a comprehensive "Connect" feature to the application, including new activities, fragments, adapters, database helpers, models, and backend integration for Connect job management and messaging. It adds new exported activities and a broadcast receiver to the manifest, multiple UI components for Connect job lists, job details, learning and delivery progress, messaging channels and chat, and payment confirmation flows. The backend integration includes network API clients for ConnectID workflows, messaging, and payment updates. Notification handling is refactored to support Connect messaging and payment actions, with a new notification channel for messaging. The codebase now includes centralized managers for ConnectID and messaging, and new interfaces for app launching and job selection. Layouts, drawables, and menu resources are added or updated to support the new UI components. Sequence Diagram(s)sequenceDiagram
participant User
participant ConnectActivity
participant ConnectManager
participant ConnectJobsListsFragment
participant ConnectJobDetailBottomSheetDialogFragment
participant ConnectMessagingActivity
participant MessageManager
participant CommCareFirebaseMessagingService
participant PaymentAcknowledgeReceiver
User->>ConnectActivity: Launch Connect feature
ConnectActivity->>ConnectManager: Initialize Connect state
ConnectActivity->>ConnectJobsListsFragment: Show job opportunities
User->>ConnectJobsListsFragment: Select job
ConnectJobsListsFragment->>ConnectJobDetailBottomSheetDialogFragment: Show job details
User->>ConnectJobDetailBottomSheetDialogFragment: Claim job / Start learning
ConnectJobDetailBottomSheetDialogFragment->>ConnectManager: Claim job / Start learning API call
ConnectManager->>ConnectActivity: Update job state/UI
User->>ConnectActivity: Open messaging
ConnectActivity->>ConnectMessagingActivity: Navigate to messaging
ConnectMessagingActivity->>MessageManager: Retrieve messages
MessageManager->>ConnectMessagingActivity: Provide messages
User->>ConnectMessagingActivity: Send message
ConnectMessagingActivity->>MessageManager: Send message to server
CommCareFirebaseMessagingService-->>ConnectMessagingActivity: Receive FCM message
CommCareFirebaseMessagingService->>MessageManager: Store message/channel
CommCareFirebaseMessagingService->>ConnectMessagingActivity: Broadcast UI update
CommCareFirebaseMessagingService-->>PaymentAcknowledgeReceiver: Receive payment acknowledgment intent
PaymentAcknowledgeReceiver->>ConnectManager: Update payment confirmation
Possibly related PRs
Suggested labels
Suggested reviewers
Tip ⚡️ Faster reviews with caching
Enjoy the performance boost—your workflow just got faster. ✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
@coderabbitai full review please |
|
Tip For best results, initiate chat on the files or code changes. @shubham1g5 I understand you're requesting a full review of this PR which introduces the "Connect" feature to CommCare Android. While I can't directly trigger CodeRabbit's full review process (that's an automated system command), I can definitely help you with specific questions about the code. From what I can see, this PR introduces a comprehensive new module with messaging functionality, job management, delivery tracking, payment processing, and learning features. The changes include new activities, fragments, adapters, models, and UI resources across many files. To get CodeRabbit's automated full review, the system should process your "@CodeRabbit full review" command automatically. Is there any specific part of the code you'd like me to look at in the meantime? For example, I can examine the architecture of the new Connect module, review specific implementation details, or discuss potential issues in particular components. 🧠 Learnings used |
|
@coderabbitai full review |
…_issue Resolved the file rename issue and matched with master file name
Use constructor for delivery details
…nto dv/connect_initial # Conflicts: # app/src/org/commcare/activities/connect/ConnectIdActivity.java
-fixes from regression qa fixes
Fixes a NPE with Connect ID Flow
…_initial 14_05 master merge in dv connect initial
Testing Code Rabbit
Labels and Review