nextcloud / nextcloud/calendar
Use FullCalendar Vue event renderer
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 1.2k
- Forks
- 332
- Avg merge
- 16h 13m
- Merged PRs (30d)
- 137
Description
Is your feature request related to a problem? Please describe.
With v3.4 we migrated to the Vue event renderer but had to revert due to critical regressions: https://github.com/nextcloud/calendar/pull/4351.
The migration is still desirable to have more control over how events are rendered. E.g. to put more icons into the element.
Describe the solution you'd like
Redo https://github.com/nextcloud/calendar/pull/4085 but pay closer attention to how it's done right. E.g. the renderer doesn't like computed properties because they can lead to endless re-render loops.
Describe alternatives you've considered
N/a
Additional context
https://github.com/nextcloud/calendar/pull/4085
https://github.com/nextcloud/calendar/issues/4235#issuecomment-1176164983
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by comparing pull requests #4085 and #4351, then read the referenced discussion in issue #4235. Rework the FullCalendar Vue event-renderer migration with attention to the renderer's handling of computed properties, and confirm that the critical regressions are avoided while enabling more control over event rendering.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend, web-dev
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100