search

Joelius300 / ChartJSBlazor

This library is a modification of the awesome ChartJs.Blazor library by mariusmuntean. It's supposed to add more functionality to the LineChart and generally make the library more complete.
Other
40 stars 6 forks source link

Added selection to which chart this applies to in issue templates. #69

Closed SeppPenner closed 5 years ago

SeppPenner commented 5 years ago

Fixes https://github.com/Joelius300/ChartJSBlazor/issues/66.

@Joelius300 Is this sufficient? Please provide me some feedback if there is something else that needs to be done.

Joelius300 commented 5 years ago

Seems good but actually I think it's sufficient if you tell them to list which charts it applies to, we don't need to mention which charts exist. For the server-side/client-side this makes sense because it's only 2 and also users might not know what possiblities there are.

I would suggest you to remove the list of charts and just say something like "Please list the charts you experienced this bug with." and "Please list the charts this feature request applies to." To both we should add "If it applies to all of them, just write 'all charts'".

SeppPenner commented 5 years ago

Okay, I can change that.

SeppPenner commented 5 years ago

I have added the labels as well (I hope, it works like that, we might need to test that). --> If you open up a new bug, the bug label should be added automatically now. (For the others as well).

SeppPenner commented 5 years ago

Ok, I have tested the generation of labels: https://github.com/Joelius300/ChartJSBlazor/issues/72. It seems to work.