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

Multiple custom fields functionality #687

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

grzegorznowacki
Copy link

IMPORTANT:

OVERVIEW:

  • Added support for custom fields with multiple values.
  • String, number and date can have now multiple values.
  • After merging cases custom fields do not disappear when their values are different.

BACKEND:

  • There is full support in Hive's backend for all types (string, number, data, boolean) when merging cases.

FRONTEND:

  • When creating case template there is full support for multiple values for string and number type.
    Date is left as single value.
  • When showing case details page there is full support for string and number.
    Date is NOT supported as multiple value on frontend side but is supported on backend side.
    After merging cases there are multiple dates in backend.
    In frontend "Multiple dates after merging. Not supported yet." message is shown and user can select new value.

@To-om To-om changed the base branch from master to develop September 20, 2018 20:12
@nadouani nadouani requested a review from To-om September 21, 2018 08:16
@zpriddy
Copy link

zpriddy commented Oct 19, 2018

I wanted to see if there were any updates on this?

@zpriddy
Copy link

zpriddy commented Oct 31, 2018

@To-om or @saadkadhi - I was wondering if there were any updates on this or if its something that is already in the develop branch?

@nadouani nadouani self-requested a review July 4, 2019 14:46
@nadouani
Copy link
Contributor

nadouani commented Jul 4, 2019

Hello, We will take a look to this PR for 3.4.0-RC2 or 3.4.0

@nadouani
Copy link
Contributor

nadouani commented Jul 4, 2019

Is there any chance to rebase this PR on top of the latest develop HEAD?

@grzegorznowacki
Copy link
Author

I resolved existing conflicts between multi-custom-field PR and develop branch. Is it now OK?

@nadouani
Copy link
Contributor

nadouani commented Jul 8, 2019

I resolved existing conflicts between multi-custom-field PR and develop branch. Is it now OK?

Thanks a lot.

@dadokkio dadokkio closed this Apr 23, 2021
@dadokkio dadokkio deleted the branch TheHive-Project:develop April 23, 2021 08:44
@nadouani nadouani reopened this Apr 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants