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

Stock: Static Template - duration per item considering individual duration rather than pages #3368

Closed
ifarzana opened this issue Mar 12, 2024 · 1 comment · Fixed by xibosignage/xibo-cms#2462
Assignees
Labels
Milestone

Comments

@ifarzana
Copy link

ID: 1835

Stock static template 2 with duration per item is applied to the item rather than the page.

Steps to reproduce:

  • Create a layout with Stock Static Template 2
  • Set 8 Stock symbols
  • Check Duration is per item
  • Set Items Per Page as 3
  • Set duration as 5 seconds

Expected:
Each page should be displayed for 5 seconds
Total layout duration should be 15seconds (for 8 items in three pages)

Actual:
Total layout duration shows 40 seconds

@ifarzana ifarzana added the bug label Mar 12, 2024
@ifarzana ifarzana added this to the 4.0.9 milestone Mar 12, 2024
@dasgarner dasgarner modified the milestones: 4.0.9, 4.0.10 Mar 18, 2024
@maurofmferrao
Copy link
Member

The problem in this case is that the template doesn't have a numItems defined, so the back-end doesn't calculate the duration for Duration Per Item. In the front-end animation, that case would change pages each 5s but still play for 40s.

dasgarner added a commit to xibosignage/xibo-cms that referenced this issue Mar 28, 2024
…Page. Also improve error responses from AlphaVantageConnector.

 fixes xibosignage/xibo#3368
dasgarner added a commit to xibosignage/xibo-cms that referenced this issue Apr 4, 2024
…Page. Also improve error responses from AlphaVantageConnector. (#2462)

fixes xibosignage/xibo#3368
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants