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

Add a CTA to the unpublished lessons notice #7374

Closed
donnapep opened this issue Dec 11, 2023 · 1 comment · Fixed by #7403
Closed

Add a CTA to the unpublished lessons notice #7374

donnapep opened this issue Dec 11, 2023 · 1 comment · Fixed by #7403
Assignees
Labels
[Pri] Normal Can wait for the next release [Status] Triaged Issue has been triaged [Type] Enhancement
Milestone

Comments

@donnapep
Copy link
Collaborator

donnapep commented Dec 11, 2023

Is your feature request related to a problem? Please describe

A notice is displayed on the frontend when previewing a course that contains no published lessons. This notice does not provide the user with next steps to take.

Note: There are actually two notices (see screenshots below). I don't know why we have notices, but I think we should be able to simplify down to a single notice.

Describe the solution you'd like

Update the notice to the following text, where clicking on "your lessons" takes the user to the Lessons page (wp-admin/edit.php?post_type=lesson) in WP admin, filtered by the current course and by Draft status.

When you're ready, let's publish your lessons in order to make them available to your students. Unpublished lessons are only visible in preview mode.

Nice-to-have: It would be nice to left-align the notice text. This isn't a mandatory requirement for this issue though. I imagine it will affect all notices (which is probably a good thing).

Additional context

Current Notice #1

Screenshot 2023-12-14 at 9 43 36 AM

Current Notice #2

Screenshot 2023-12-14 at 9 44 34 AM

Desired Notice

Screenshot 2023-12-11 at 12 11 01 PM

@renatho
Copy link
Contributor

renatho commented Dec 22, 2023

Note: There are actually two notices (see screenshots below). I don't know why we have notices, but I think we should be able to simplify down to a single notice.

I think we'll need both anyway because of the notice for the students. For them, I imagine the "Current Notice 1" is probably enough for any situation, because it's just informative and they don't need to take any action.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Pri] Normal Can wait for the next release [Status] Triaged Issue has been triaged [Type] Enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants