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.

Sign up for GitHub

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

Jump to bottom

Fix dashboard edit action for scheduled listings #2872

Open
wants to merge 2 commits into
base: trunk
Choose a base branch
from
Open

Fix dashboard edit action for scheduled listings #2872

wants to merge 2 commits into from

Conversation

yscik
Copy link
Contributor

@yscik yscik commented Jan 30, 2025 •
edited
Loading

Changes Proposed in this Pull Request

  • Handle the future post status in the job dashboard, show the "Edit" action can show up for scheduled listings
  • Use the "Allow published edits" settings instead of "Allow editing of pending listings" for deciding if a scheduled job should be editable

Testing Instructions

  • Make sure Settings → Submissions → Allow Published Edits is enabled (with or without re-approval)
  • Create a new listing, set a scheduled date
  • Approve it as admin
  • Check that it's editable in the job dashboard
image

Release Notes

  • Fix: Enable editing of scheduled listings if published/approved listings are editable

Screenshot / Video

Plugin build for 76d5226
📦 Download plugin zip
▶️ Open in playground

@yscik yscik force-pushed the fix/edit-scheduled branch from b400b88 to 76d5226 Compare January 30, 2025 09:58
@yscik yscik self-assigned this Jan 30, 2025
@yscik yscik requested a review from a team January 30, 2025 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@yscik yscik

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@yscik