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

LastFinished Sort: Use customDate, use updatedAt as fallback #494

Merged
merged 1 commit into from
Apr 20, 2024

Conversation

IRHM
Copy link
Member

@IRHM IRHM commented Apr 20, 2024

Changes made

Sort by Last Finished not taking custom dates into account.

Fixes #493

@IRHM IRHM added the bug Hmm, this isn't working correctly label Apr 20, 2024
@IRHM IRHM self-assigned this Apr 20, 2024
@IRHM IRHM merged commit 19c4bc4 into dev Apr 20, 2024
1 check passed
@IRHM IRHM deleted the fix-sort-lastfin branch April 20, 2024 17:00
truecharts-admin referenced this pull request in truecharts/public Apr 23, 2024
….0@9c5d999 by renovate (#21201)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [ghcr.io/sbondco/watcharr](https://github.com/sbondCo/Watcharr) |
minor | `v1.38.2` -> `v1.39.0` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>sbondCo/Watcharr (ghcr.io/sbondco/watcharr)</summary>

###
[`v1.39.0`](https://github.com/sbondCo/Watcharr/releases/tag/v1.39.0)

[Compare
Source](https://github.com/sbondCo/Watcharr/compare/v1.38.2...v1.39.0)

#### 🧠 New

- Support pinning watched list items to the top of your list (pinned
items are highlighted with a gold outline) by
[@&#8203;IRHM](https://github.com/IRHM) in
[https://github.com/sbondCo/Watcharr/pull/498](https://github.com/sbondCo/Watcharr/pull/498)

![image](https://github.com/sbondCo/Watcharr/assets/37304121/005eb3c4-ea90-428b-8687-91bb729a85a8)
- Support emby branding (new `USE_EMBY` setting that will swap out
Jellyfin branding for Embys branding, no auth logic has been changed) by
[@&#8203;IRHM](https://github.com/IRHM) in
[https://github.com/sbondCo/Watcharr/pull/504](https://github.com/sbondCo/Watcharr/pull/504)

#### 💯 Changed

- Better thoughts (now a modal is opened so you can add your thoughts
more easily) by [@&#8203;IRHM](https://github.com/IRHM) in
[https://github.com/sbondCo/Watcharr/pull/495](https://github.com/sbondCo/Watcharr/pull/495)
- Improve search (add content type filters and remove dumb sort) by
[@&#8203;IRHM](https://github.com/IRHM) in
[https://github.com/sbondCo/Watcharr/pull/496](https://github.com/sbondCo/Watcharr/pull/496)

![image](https://github.com/sbondCo/Watcharr/assets/37304121/6c6a846d-da1e-4061-a2b0-e7839fd87176)
- Support detailed view for posters on search page by
[@&#8203;IRHM](https://github.com/IRHM) in
[https://github.com/sbondCo/Watcharr/pull/497](https://github.com/sbondCo/Watcharr/pull/497)

#### 🏗️ Fixed

- LastFinished Sort: Prefer custom dates that are set on `FINISHED`
activities by [@&#8203;IRHM](https://github.com/IRHM) in
[https://github.com/sbondCo/Watcharr/pull/494](https://github.com/sbondCo/Watcharr/pull/494)
- Content Pages: Make btns container wrap elements when overflowing by
[@&#8203;IRHM](https://github.com/IRHM) in
[https://github.com/sbondCo/Watcharr/pull/499](https://github.com/sbondCo/Watcharr/pull/499)

#### 🥇 Credit

A big thanks to [@&#8203;n00b12345](https://github.com/n00b12345),
[@&#8203;simonbcn](https://github.com/simonbcn),
[@&#8203;mommyune](https://github.com/mommyune),
[@&#8203;gardebreak](https://github.com/gardebreak) and
[@&#8203;rguinn829](https://github.com/rguinn829) for suggesting the
improvements/fixes made in this release!

#### 🆘 Getting Help

If you need help, encounter an issue or find a bug please [create an
issue](https://github.com/sbondCo/Watcharr/issues/new/choose) or [join
our space on Matrix](https://matrix.to/#/#watcharr:matrix.org) for
support. Always happy to help!

**Package**:
https://github.com/orgs/sbondCo/packages/container/watcharr/207084376?tag=v1.39.0
or [on docker
hub](https://hub.docker.com/layers/sbondco/watcharr/v1.39.0/images/sha256-9c5d999d92d3475b6a91317727d1620f14b5d85309a3878604f38c60eee22526?context=explore)
**Full Changelog**:
sbondCo/Watcharr@v1.38.2...v1.39.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://github.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zMTkuMCIsInVwZGF0ZWRJblZlciI6IjM3LjMxOS4wIiwidGFyZ2V0QnJhbmNoIjoibWFzdGVyIiwibGFiZWxzIjpbImF1dG9tZXJnZSIsInVwZGF0ZS9kb2NrZXIvZ2VuZXJhbC9ub24tbWFqb3IiXX0=-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Hmm, this isn't working correctly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sort by Last Finished doesn't take custom dates into account
1 participant