Repository navigation
Request: add active flag support for ConstructorItem #170
Description
Add active flag support
- Introduce a new
activeboolean property inConstructorItem. - Ideally a new signature when initialising the
ConstructorItem - The default could be true.
E.g.
public ConstructorItem(String id, String name, Boolean active) throws IllegalArgumentException {
if (id == null) {
throw new IllegalArgumentException("id is required");
}
this.id = id;
this.name = name;
this.active = active;
this.suggestedScore = null;
this.keywords = null;
this.url = null;
this.imageUrl = null;
this.description = null;
this.facets = null;
this.metadata = null;
this.groupIds = null;
}p.s. I am raising this issue because I couldn't verify if the active is by default true for createOrReplaceItems endpoint. I also couldn't not find in the sdk the logic for it. Lastly, I tried to contribute and I could not push my branch into this repo :)
Activity
esezen commentedon Oct 20, 2025
Hey @ahmedmaazin , thanks for opening this issue. We don't have any special logic for
activefield at the moment but you can still put it into yourmetadataobject and it will work as expected. However, I understand it would be more convenient if it was a field that you can set similar todescription. Let me look into this and open up a PR. In the meantime, if you would like to contribute, you can always fork this repository and open up a PR that way. Let me know if that works!esezen commentedon Oct 20, 2025
Hey @ahmedmaazin , I created a PR here: #171
It does not change the constructor but it gives you the getters/setters to set it. If setting this field when initializing the class is important for you, please let me know and we will discuss it internally to see if we can support it
ahmedmaazin commentedon Oct 20, 2025
Hey thanks a lot, I did not see this comment/pr at the time and went ahead and pushed my branch with the desired changes. Let me take a look what you have added :)
ahmedmaazin commentedon Oct 20, 2025
It is actually not important to set it when I initialise the ConstructorItem if I can later use the setActive() method when I need it.
esezen commentedon Oct 23, 2025
Hey @ahmedmaazin , the change has been merged and it's available in the latest version of the library. Thanks again for the suggestion!