search

OpenSourceOrg / dotOrg

Public tracker of opensource.org issues
GNU General Public License v3.0
7 stars 1 forks source link

SugarCalendar: Recurrence shows the wrong information on the front end #55

Open smaffulli opened 4 months ago

smaffulli commented 4 months ago

Description

When setting up the recurrence to every other week one would expect the information on the front end to be shown correctly.

I set this event to recur every two weeks:

image

but the frontend shows the event repeating weekly, which is wrong.

image

Steps to reproduce

No response

What you expected to happen

No response

What actually happened

No response

nickpagz commented 4 months ago

It looks like the Advanced Recurring extension doesn't provide any output formatting options for the more complex recurring patterns, such as repeating events with "every" 2 or 3 weeks/months etc intervals.

I've reached out to Sugar Calendar support to ask for some help or direction, albeit without the license key used to purchase the extension, so I'm not sure they'll help.

In the meantime I've come up with my own version of the formatting function that tries to give us a useable format for some of the cases. This can be seen in PR #63 Please note the exceptions/cases not currently covered in the PR description.

nickpagz commented 3 months ago

@smaffulli So based on the response from Sugar Calendar it doesn't look like they've got an ETA for a fix on this. I've also reviewed their work-around, and I think this won't work either as modifying the translation string could/would affect all other events on the site.

I can continue working on the PR I started (mentioned in the previous comment), but since this will use a custom function for generating the text, if/when a fix comes out from Sugar Calendar the site won't use it until we revert the changes made in the PR.

I think it boils down to how urgent this is. Do you have existing events where you need this functionality? Or can it wait? I'm completely open to whatever option you choose, and happy to work on progressing the PR if needed.

Thanks!

smaffulli commented 3 months ago

This can wait. We've asked concierge also to suggest alternative tools because Sugar also has issues with registrations to events, it doesn't support free events.

On Wed, Mar 6, 2024, 12:56 Nicholas Pagazani @.***> wrote:

@smaffulli https://github.com/smaffulli So based on the response from Sugar Calendar it doesn't look like they've got an ETA for a fix on this. I've also reviewed their work-around, and I think this won't work either as modifying the translation string could/would affect all other events on the site.

I can continue working on the PR I started (mentioned in the previous comment), but since this will use a custom function for generating the text, if/when a fix comes out from Sugar Calendar the site won't use it until we revert the changes made in the PR.

I think it boils down to how urgent this is. Do you have existing events where you need this functionality? Or can it wait? I'm completely open to whatever option you choose, and happy to work on progressing the PR if needed.

Thanks!

— Reply to this email directly, view it on GitHub https://github.com/OpenSourceOrg/dotOrg/issues/55#issuecomment-1980709851, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKJJAGU74FMAH4RKMKRGBDYW4AATAVCNFSM6AAAAABC5Q5D32VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBQG4YDSOBVGE . You are receiving this because you were mentioned.Message ID: @.***>