openlibhums / openlibhums/janeway

Using the exclude operator on journal carousels can result in a server error

Open
#4,172 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug Size M
Dominant language
Python
Stars
238
Forks
97
Avg merge
9d 1h
Merged PRs (30d)
8

Description

Describe the bug
When you utilise the exclude feature of the carousel and add articles to the exclude field this will cause the carousel to break.

Janeway version
1.5.3/1.6

To Reproduce
Steps to reproduce the behavior:

  1. Add carousel
  2. Mark Exclude
  3. Add articles to the selected field

Expected behavior
Carousel should render excluding those articles.

Note: Locally the carousel simply fails to load, in production a server error is returned.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Reproduce the failure by creating a journal carousel, enabling Exclude, and adding articles to the selected field. Start with the local carousel failure and its server error details; done means the carousel renders while omitting the excluded articles.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.