[GH-ISSUE #78] Allow to disable records #45

Closed
opened 2026-02-28 01:20:43 +03:00 by kerem · 2 comments
Owner

Originally created by @nickygerritsen on GitHub (Mar 19, 2016).
Original GitHub issue: https://github.com/tuxis-ie/nsedit/issues/78

First of all, great product. I really enjoy it and its simplicity.

However, would it be possible to add the option to disable a record? As far as I know this would only mean we need to set "disabled: true" when sending it to the API.

The biggest problem I currently see is that we use the whole JSON as ID for the record and that might mean the ID changes when we disable it, but maybe I'm wrong in that sense?

Disabling records would make sense for us as we sometimes want to temporarily not serve some records.

If the ID thing is not a problem, I think I might even easily be able to provide a PR and am willing to do that :).

Originally created by @nickygerritsen on GitHub (Mar 19, 2016). Original GitHub issue: https://github.com/tuxis-ie/nsedit/issues/78 First of all, great product. I really enjoy it and its simplicity. However, would it be possible to add the option to disable a record? As far as I know this would only mean we need to set "disabled: true" when sending it to the API. The biggest problem I currently see is that we use the whole JSON as ID for the record and that might mean the ID changes when we disable it, but maybe I'm wrong in that sense? Disabling records would make sense for us as we sometimes want to temporarily not serve some records. If the ID thing is not a problem, I think I might even easily be able to provide a PR and am willing to do that :).
kerem 2026-02-28 01:20:43 +03:00
Author
Owner

@tuxis-ie commented on GitHub (Mar 21, 2016):

I don't think there would be an issue. Feel free to fix a pr and I'd be happy to merge it.

<!-- gh-comment-id:199144646 --> @tuxis-ie commented on GitHub (Mar 21, 2016): I don't think there would be an issue. Feel free to fix a pr and I'd be happy to merge it.
Author
Owner

@tuxis-ie commented on GitHub (Mar 21, 2016):

Merged

<!-- gh-comment-id:199180908 --> @tuxis-ie commented on GitHub (Mar 21, 2016): Merged
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/nsedit#45
No description provided.