-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Feature: calendar widget #2077
Feature: calendar widget #2077
Conversation
Signed-off-by: Denis Papec <[email protected]>
Signed-off-by: Denis Papec <[email protected]>
Signed-off-by: Denis Papec <[email protected]>
Signed-off-by: Denis Papec <[email protected]>
Signed-off-by: Denis Papec <[email protected]>
Holy crap this is awesome, when is this coming to the main branch? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great overall and really 'just works' out of the box.
I hit a couple tiny bugs at first so fixed those and I made small changes, all individual commits so you can see (we'll squash & merge this anyway).
Only notable change is making it scale width on larger sizes and I made the 'includeSeries' and 'includeAuthor' options default to true, my feeling is it provides a better experience and users can always turn that off if they want.
Great work, let me know if you have any concerns re the above
Oh yea, I also changed |
Signed-off-by: Denis Papec <[email protected]>
Those changes looks good, thank you 👍 . |
Sorry, what did you mean by your last comment? You want me to revert your last commit? 14e9ce5 |
(because |
Sorry, I was in a rush this morning. |
This reverts commit 14e9ce5.
Ah yea, I see. Honestly, Im ok without that anyway, lets merge it 💪 |
@denispapec would it be possible to implement a button that will go to the current date? For example, by clicking on the "Calendar" we could go back to the current date. This is useful once you go back or forward a few months and want to go back. Workaround for this is to just refresh the page. |
@denispapec , I posted a new feature request here: #2086 Thanks for this awesome widget |
Commenting on closed PRs is not the way to make a feature or get help. to answer your q, those links in the screenshot are just other services in a column next to the calendar |
Proposed change
This PR implements calendar widget, with optional integrations to show events from supported widgets. This PR has support for radarr, sonarr, lidarr and readarr.
Closes #1243
Closes #1015
Closes #1200
Type of change
Checklist:
pnpm lint
.