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

Sample Data: When to show? #3224

Closed
maurofmferrao opened this issue Nov 8, 2023 · 6 comments
Closed

Sample Data: When to show? #3224

maurofmferrao opened this issue Nov 8, 2023 · 6 comments
Assignees
Labels
enhancement An improvement to an existing feature
Milestone

Comments

@maurofmferrao
Copy link
Member

We're currently showing sample data even when a module is properly configured (e.g. Calendar widget with no events returned ).

Sample Data only makes sense to be shown when a module/widget is invalid.

@maurofmferrao maurofmferrao added the enhancement An improvement to an existing feature label Nov 8, 2023
@maurofmferrao maurofmferrao added this to the 4.0.6 milestone Nov 8, 2023
@maurofmferrao maurofmferrao self-assigned this Nov 8, 2023
@rubenpingol-xibo
Copy link

To be clear here, when the module is properly configured (e.g. Calendar widget), this means it has a valid Feed URL. And when there is no event from the provided Feed URL, there should be no items to be shown as well in the calendar. Is this correct?

We will only show Sample Data only when;

  1. The module is not properly configured (e.g. Calendar widget with no Feed URL provided)
  2. Provided with invalid Feed URL or the module is invalid
    This can mean that the response from the backend for the widget data could be something like
{
  success: false,
  error: 0,
  message: "The iCal provided is not valid, please choose a valid feed\n"
}

@maurofmferrao
Copy link
Member Author

Yes, that's correct. We only want to show Sample Data when the widget is not configured ( isValid is false ). For calendar it makes sense to have it like this. Not sure about something like RSS if the feed is valid but returns no items, since the interface will not show anything, not sure if we need to mark isValid as false in that case @dasgarner ?

@rubenpingol-xibo
Copy link

Alright, thanks for confirming. How about for RSS when the feed is valid but no items, what should we display on the editor and on preview?

@dasgarner
Copy link
Member

I've opened a discussion on this so we can work out what we want to do.

rubenpingol-xibo added a commit to rubenpingol-xibo/xibo-cms that referenced this issue Nov 21, 2023
rubenpingol-xibo added a commit to rubenpingol-xibo/xibo-cms that referenced this issue Nov 22, 2023
rubenpingol-xibo added a commit to rubenpingol-xibo/xibo-cms that referenced this issue Nov 23, 2023
rubenpingol-xibo added a commit to rubenpingol-xibo/xibo-cms that referenced this issue Nov 26, 2023
dasgarner pushed a commit to xibosignage/xibo-cms that referenced this issue Nov 27, 2023
* Widgets: Add logic for when to show sample data in the editor/preview/player
relates to xibosignage/xibo#3224
rubenpingol-xibo added a commit to rubenpingol-xibo/xibo-cms that referenced this issue Nov 28, 2023
rubenpingol-xibo added a commit to xibosignage/xibo-cms that referenced this issue Nov 28, 2023
rubenpingol-xibo added a commit to rubenpingol-xibo/xibo-cms that referenced this issue Nov 29, 2023
rubenpingol-xibo added a commit to xibosignage/xibo-cms that referenced this issue Nov 29, 2023
@maurofmferrao
Copy link
Member Author

@rubenpingol-xibo I just found that Attribution for Weather is behaving in a different way than the other elements in the same widget

Image

@dasgarner
Copy link
Member

✅ tested

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An improvement to an existing feature
Projects
None yet
Development

No branches or pull requests

3 participants