-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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 schedule posts #8509
Open
Rajat-Dabade
wants to merge
18
commits into
main
Choose a base branch
from
feature_schedule_posts
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Feature schedule posts #8509
Conversation
This file contains 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
* Added types and model for scheduled post * updated model for scheduled post * updated types * Fixed comment --------- Co-authored-by: Harshil Sharma <[email protected]>
/update-branch |
* WIP * Separate component for core and custom options * WIP * WIP * WIP * unified options into single component * Scheduled post menu ready * Displayed date time picker by default * WIP * Lint fix * i18n fix * removed unused screen * Added user time format preference * review fixex and improvements * Renamed a file
* WIP * Separate component for core and custom options * WIP * WIP * WIP * unified options into single component * Scheduled post menu ready * Displayed date time picker by default * WIP * Lint fix * i18n fix * removed unused screen * API call sent successfully * Added user time format preference * review fixex and improvements * Handled loading sttae * Handled loading sttae * Handled mentions and files and priority * Cleanup * lint fix * i18n fix * WIP * Displayed error snackbar * Fixed time used for debugging * Lint fix * review fixex and improvements * Renamed a file * Adding tests * Adding tests * lint fix * cleanup * Added draft_input tests * test: Add comprehensive tests for SendAction component * Added send_action tests * lint fix * test: Add comprehensive test coverage for useHandleSendMessage hook * test: Add test for scheduled post creation in send message hook * fix: Mock createScheduledPost function in test to resolve matcher error * WIP tests * test: Add tests for SnackBar component * Added snack bar tests * Cleanup * test: Add empty test file for scheduled post picker * test: Add tests for ScheduledPostOptions component * test: Improve ScheduledPostOptions test coverage and error handling * test: Update ScheduledPostOptions test suite with new database and timezone mocking * test: Fix timeout issues in ScheduledPostOptions test suite * Added scheuidled post picker tests * test: Add empty test file for scheduled post footer * test: Add comprehensive tests for ScheduledPostFooter component * feat: Add animatedFooterPosition prop to ScheduledPostFooter test * test: Add tablet rendering test for ScheduledPostFooter * test: Update ScheduledPostFooter test mocks and remove unused imports * test: Add mocks for BottomSheet context in scheduled post footer tests * Added scheudled post footer test * test: Add comprehensive tests for handle_send_message hook * Enhanced handle send message tests * cleanup * Added missing snapshots * test: Add error logging and force logout validation for scheduled post tests * Updated test * used toBeVisible instead of toBeTruthy * test: Update test assertions to use toHaveBeenCalledWith * test: Add assertions to prevent sendMessage when send button is disabled * test: Replace snapshot tests with explicit UI element checks * test improvements * Deleted unused snapshots * Updated snaopshot * feat: Add close button ID for scheduled post options bottom sheet * test: Add mock scheduling flow to scheduled post options test * Fixed incorrect test * scheudled post test enhancement * minimised a test * Scheduled post indicator (#8522) * WIP * WIP * DFisplayed everything correctly * Lint fix * handled thread and padding * removed a comment * Fixed i18n * test: Add empty test file for scheduled post indicator component * test: Add tests for ScheduledPostIndicator component * test: Rewrite ScheduledPostIndicator tests with improved coverage * test: Update scheduled post indicator test setup with TestHelper * fix: Replace non-existent addPreferencesToServer with TestHelper.setPreference * refactor: Update ScheduledPostIndicator tests to use operator.handleConfigs * refactor: Use renderWithEverything and consistently pass database prop * Added tests for scheduled_post_indicator * test: Add channel screen test file * test: Add comprehensive tests for Channel component * refactor: Remove unused import and clean up whitespace in channel test file * refactor: Update Channel component tests with database and renderWithEverything * refactor: Update Channel test cases to remove async/await and add type * test: Wrap channel screen render in act() to handle async updates * fix: Resolve unmounted renderer error in Channel snapshot test * test: Remove unnecessary async act in channel test snapshot * fix: Resolve unmounted renderer issue in channel screen test * Added chanels test * test: Add empty test file for thread screen * test: Add tests for Thread component with rendering and call feature scenarios * cleanup * Mock up0date * temperorily removed a flaky test * review fixes * Fixed a test * Fixed a test * Fixed a test
* Added types, models and transformer for scheduled post * added handler for receiving all the scheduled post * Added migration script for adding schedule post table * version upgraded * Added schedule post model in Database Manager * reverted back unnecessary changes * Added schema for schedule post table * Updated database schema tests * Addressed review comments * Added test for handleScheduledPost * Added test for schedule post transformer * removed undefined properties * reverted unnecessary changes * Updated logic and test case for fetching records according to teams * remove unnecessary changes * minor * Updated logic * Updated logic for handling scheduled post * Updated interface for handleScheduledPosts * Removed extrac processRecord when deleting the schedule post * Update test case * Updated query for fetching schedule post from local db * Updated logic when review all the scheduled post * ci fixes
…ost (#8526) * added handler for receiving all the scheduled post * Added migration script for adding schedule post table * reverted back unnecessary changes * Addressed review comments * Added test for handleScheduledPost * removed undefined properties * reverted unnecessary changes * Updated logic and test case for fetching records according to teams * remove unnecessary changes * Updated logic for handling scheduled post * Update test case * Updated logic when review all the scheduled post * ci fixes * Schedule post rest client, local/remote/websocket handling * review fixes * Added test for local client * Added test case for handle delete scheduled post * Added test for remote client * Added test for websocket schedule post events * Added test for scheduled_post rest client * Added include direct channel post flag for handling schedule post fetch request * Updated test case * Addressed review comments * instead of mocking adding it to database * Updated test * import fixes * addressed review comments * minor * removed redundent checks * coverage --------- Co-authored-by: Harshil Sharma <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Do Not Merge/Awaiting PR
Awaiting another pull request before merging (e.g. server changes)
release-note
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.
Summary
Main PR for Feature Schedule Posts
Ticket Link
Checklist
E2E iOS tests for PR
.Device Information
This PR was tested on:
Screenshots
Release Note