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

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

Closed
smaffulli opened this issue Feb 7, 2024 · 4 comments
Closed

Comments

@smaffulli
Copy link
Contributor

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
Copy link
Collaborator

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
Copy link
Collaborator

nickpagz commented Mar 6, 2024

@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
Copy link
Contributor Author

smaffulli commented Mar 6, 2024 via email

@ajolo2ky
Copy link
Contributor

ajolo2ky commented Jul 2, 2024

Closing as we are no longer using SugarCalendar

@ajolo2ky ajolo2ky closed this as completed Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants