Skip to content

Carnival

Compare
Choose a tag to compare
@LMSCloudPaulD LMSCloudPaulD released this 07 Nov 22:20
· 46 commits to master since this release

What's Changed

Fixes

  • Infinite "Scrolling": Fixed the infinite "scrolling" (rather requesting) functionality for the LMSEventsView component. The pagination of event results now dynamically adjusts with the _per_page parameter, removing the need for a separate event count endpoint. This update provides a smoother user experience by seamlessly loading more events as needed.
  • Event Count Endpoint Removal: The /public/events/count endpoint and its associated backend method count have been removed to optimize the event fetching process in line with the new infinite scrolling feature.

Chores

  • Added caching for perltidy to improve the performance of formatting operations.
  • Included individual entries for formatting in the build process to cater to different file types.
  • Adjusted the build package script to use canonical exit values for better interoperability.
  • Updated the .gitignore to include cache files for prettier and perltidy.

Refactoring and Improvements

  • Refactored the event retrieval logic in the Public/Events controller for better readability and maintainability.
  • Fixed issues related to event filtering and target group selection to ensure accurate data retrieval.
  • Enhanced DOM update performance by using the repeat directive in the frontend.

Breaking Changes

  • The removal of the count method and /public/events/count endpoint requires clients to update their implementations to handle event counts via the infinite scrolling mechanism. This change is intended to streamline the frontend experience but necessitates changes on the client side.

Reversions

  • Reverted a previous fix where id was temporarily set as the default sort column due to unforeseen issues that arose from the change.

Miscellaneous

  • Corrected the name of perltidyrc as it was accidentally named perlcriticrc.
  • Fixed an issue where settings updates were not correctly refetched and rendered, which caused confusion due to outdated state information.