Skip to content

Show message when there are no content in the page#12056

Merged
alecslupu merged 2 commits intodevelopfrom
fix/page-no-content
Nov 25, 2023
Merged

Show message when there are no content in the page#12056
alecslupu merged 2 commits intodevelopfrom
fix/page-no-content

Conversation

@andreslucena
Copy link
Copy Markdown
Member

🎩 What? Why?

When there are no content in a page, we don't show any message, just a blank page.

This PR changes it to show a message.

I've found about this error while reviewing #11480.

📌 Related Issues

Testing

  1. Create a new process
  2. Create a pages component
  3. Click in the "Preview" icon in the admin's component page
  4. See the page

📷 Screenshots

Screenshot of the no content page in pages module

♥️ Thank you!

@andreslucena andreslucena added module: pages type: fix PRs that implement a fix for a bug labels Nov 22, 2023
github-actions[bot]
github-actions bot previously approved these changes Nov 22, 2023
@alecslupu
Copy link
Copy Markdown
Contributor

@andreslucena this is draft ...

@andreslucena andreslucena marked this pull request as ready for review November 22, 2023 16:04
@andreslucena
Copy link
Copy Markdown
Member Author

@andreslucena this is draft ...

I forget to push the button, now it's ready for a review

Copy link
Copy Markdown
Contributor

@alecslupu alecslupu left a comment

Choose a reason for hiding this comment

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

👍

@alecslupu alecslupu merged commit 398d43c into develop Nov 25, 2023
@alecslupu alecslupu deleted the fix/page-no-content branch November 25, 2023 08:24
entantoencuanto added a commit that referenced this pull request Nov 29, 2023
…/redesign-staging

* fix/control-comments-replies-polymorphism:
  Add a polymorphic option to restrict acts_as_tree lookup
  Add tests to check the 'Hide replies' message error
  Fix pipeline after translations were updated (#12084)
  Remove the duplicate reference of icons (#12083)
  Fix modal forms for editor (#12077)
  New Crowdin updates (#12003)
  Display only once resource type in search (#12041)
  Redesign: fix responsiveness on nav-bar admin (#12045)
  Apply flash styles to Announcements (part 2) (#12039)
  Remove legacy design support from WYSIWYG editor (TipTap) (#12015)
  Extract methods from meetings seeds (#12053)
  Show message when there are no content in the page (#12056)
  Show message when there are no sortitions (#12057)
  Fix icons on flash alerts (#12069)
  Update numbering in RELEASE_NOTES.md (#12066)
  Add prerequirements to the manual installation tutorial (#12068)
  Differentiate non-clickable elements in admin menu (#12000)
  Remove official logo header image (#12064)
@alecslupu alecslupu added this to the 0.28.0 milestone Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: pages type: fix PRs that implement a fix for a bug

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants