Skip to content
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

Added local remote and websocket action for updating scheduled post #8600

Open
wants to merge 13 commits into
base: scheduled_post_tabs
Choose a base branch
from

Conversation

Rajat-Dabade
Copy link
Contributor

Summary

Ticket Link

Checklist

  • Added or updated unit tests (required for all new features)
  • Has UI changes
  • Includes text changes and localization file updates
  • Have tested against the 5 core themes to ensure consistency between them.
  • Have run E2E tests by adding label E2E iOS tests for PR.

Device Information

This PR was tested on:

Screenshots

Release Note


@Rajat-Dabade Rajat-Dabade added the Do Not Merge Should not be merged until this label is removed label Feb 18, 2025
@Rajat-Dabade Rajat-Dabade requested review from larkox, enahum and harshilsharma63 and removed request for enahum February 21, 2025 07:15
@Rajat-Dabade Rajat-Dabade self-assigned this Feb 21, 2025
@Rajat-Dabade Rajat-Dabade marked this pull request as ready for review February 21, 2025 07:15
@Rajat-Dabade Rajat-Dabade requested a review from enahum February 21, 2025 07:15
@@ -32,6 +39,28 @@ export async function createScheduledPost(serverUrl: string, schedulePost: Sched
}
}

export async function updateScheduledPost(serverUrl: string, scheduledPost: ScheduledPost, connectionId?: string, fetchOnly = false) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess this can be combined with the other action as is almost the same code just the rest endpoint but 0/5

@Rajat-Dabade Rajat-Dabade force-pushed the updating-scheduled-post-actions branch from 20d7c19 to 68e9542 Compare February 25, 2025 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do Not Merge Should not be merged until this label is removed release-note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants