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

fix: Recurrent events only one event is displayed on exchange agenda- EXO-68948 #97

Merged
merged 2 commits into from
Jan 22, 2024

Conversation

GouadriaHanen
Copy link
Contributor

@GouadriaHanen GouadriaHanen commented Jan 22, 2024

before this change, when a recurrent event is added only one event is added to the exchange data without any recurrence info causing the non-display of all the events
After this change, the recurrence info is added to the event and all recurrent events are displayed

Copy link

sonarcloud bot commented Jan 22, 2024

@GouadriaHanen GouadriaHanen merged commit 3916b61 into feature/maintenance Jan 22, 2024
4 checks passed
@GouadriaHanen GouadriaHanen deleted the EXO-68948 branch January 22, 2024 16:30
@rdenarie
Copy link
Member

@GouadriaHanen In your commit message, you do not explain WHY it is not working nor how you fix it.
You only explain what is the symptome of the problem
If you imagine in 6 month, you will have difficulties to remember what is the fix :)

Can you ensure in future fix to better explain your commit ?
Thanks a lot

GouadriaHanen added a commit that referenced this pull request Jan 23, 2024
…XO-68948 (#97)

before this change, when a recurrent event is added only one event is added to the exchange data without any recurrence info causing the non-display of all the events
After this change, the recurrence info is added to the event and all recurrent events are displayed
GouadriaHanen added a commit that referenced this pull request Jan 23, 2024
…XO-68948 (#97)

before this change, when a recurrent event is added only one event is added to the exchange data without any recurrence info causing the non-display of all the events
After this change, the recurrence info is added to the event and all recurrent events are displayed
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

Successfully merging this pull request may close these issues.

3 participants