Skip to content
This repository has been archived by the owner on Mar 11, 2021. It is now read-only.

Generic field support #2070

Open
kwk opened this issue Apr 30, 2018 · 2 comments
Open

Generic field support #2070

kwk opened this issue Apr 30, 2018 · 2 comments

Comments

@kwk
Copy link
Collaborator

kwk commented Apr 30, 2018

Currently the backend has places in which work item fields are treated by their name and not their type. This makes it hard to add support for fields that are similar in type but have a different name. This is especially true for a lot of fields from the new scrum template. To give an example: we can perfectly handle a work item's description field using markdown and preview, but we cannot handle a field like steps-to-reproduce in a Bug work item, simply because it is different.

@jarifibrahim
Copy link
Member

To give an example: we can perfectly handle a work item's description field using markdown and preview, but we cannot handle a field like steps-to-reproduce in a Bug work item, simply because it is different.

@kwk Why can't we handle steps-to-reproduce similar to description? Steps-to-reproduce would be part of description, right? It isn't a seperate entity.

Can you please add another example of why do we need this change? I do not understand.

@kwk
Copy link
Collaborator Author

kwk commented May 2, 2018

@jarifibrahim essentially any field that has a "complicated" type (e.g. lists and markdown fields) is currently handled by name and not by type. So all the special behavior is solely driven by a field's name which makes it impossible to re-use that logic in fields with the same type.

And no, the steps-to-reproduce is actually a standalone field.

kwk added a commit that referenced this issue May 2, 2018
… type (#2069)

In this PR I've changed the signature of the `controller/ConvertWorkItem` and `controller/ConvertWorkItems` function. They now accept the work item type of the work item that is supposed to be converted. The type itself is **not yet** used but it will in a future PR.

This is needed to support generic fields (see #2070).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants