Add '1 Week' Option to Past Horizon setting

Created on 29 December 2023, 11 months ago
Updated 1 April 2024, 8 months ago

Problem/Motivation

The Google Calendar module currently allows users to set a "Past horizon" for importing events, which determines how far back in time the module will go to import calendar events. This setting also influences the event cleanup policy. However, the module lacks a crucial time frame option – '1 week'. This limitation restricts users who need to import and manage events on a weekly basis, which is a common scenario for many organizations and individuals using Drupal. By not having a '1 week' option, users may either miss important recent events or have to deal with an excessively large time horizon, impacting the efficiency and relevance of the calendar data.

Steps to reproduce

Navigate the Events Settings page on the module (/admin/config/google_calendar/events). Scroll down to Events Refresh section. Please note that 'Past horizon' field has not the '1 week' option.

Proposed resolution

The proposed resolution is to extend the "Past horizon" field options by adding a '1 week' selection to that form. This addition will provide greater flexibility and usability for the module, catering to users who operate on a weekly event cycle.

Feature request
Status

Fixed

Version

2.5

Component

Code

Created by

🇧🇷Brazil raprieto

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @raprieto
  • Status changed to Needs review 11 months ago
  • 🇧🇷Brazil raprieto

    I have updated the Google Calendar module to include the proposed '1 week' option in the Past horizon field, as detailed in the issue. This modification aims to enhance the module's functionality by allowing users to manage events on a weekly basis, which aligns with common event management practices.

    The corresponding changes have been implemented and are available in the merge request: [Merge Request #7](https://git.drupalcode.org/project/google_calendar/-/merge_requests/7). I am moving the status of this issue to "Needs Review" to invite feedback, testing, and review from the community and maintainers. This review will ensure that the new feature integrates seamlessly with the module and functions as intended across different use cases.

    I look forward to any comments or suggestions and am ready to make further adjustments if needed to align with the module's standards and user requirements.

  • Status changed to RTBC 11 months ago
  • 🇦🇷Argentina tguerineau

    I have completed testing the '1 Week' option in the 8.x-2.x branch of the Google Calendar module, and I'm pleased to report that it works as expected.

    My testing involved integrating a Google Calendar with the module and creating a variety of test events, some within the past week and some older. After selecting the new '1 Week' option in the "Past horizon" field, I observed that the module correctly imported events from the past week, while excluding older events.

  • Status changed to Fixed 8 months ago
  • 🇬🇧United Kingdom rivimey

    Thanks.

  • Issue was unassigned.
  • 🇬🇧United Kingdom rivimey
  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024