Remove old `data` property for column + stacked charts
requested to merge 1020-remove-old-data-property-in-favour-of-bars-and-lines-for-column-stacked-charts into master
What does this MR do?
This is a follow up MR in relation to #889 (closed) :
-
!1703 (merged) added the
bars
andlines
properties - gitlab!46508 (merged) updated the charts in the gitlab project
This MR removes the data
prop that is no longer needed.
Screenshots
No visual changes to the components
Does this MR meet the acceptance criteria?
Conformity
-
Code review guidelines. -
GitLab UI's contributing guidlines. -
If it changes a Pajamas-compliant component's look & feel, the MR has been reviewed by a UX designer. -
If it changes GitLab UI's documentation guidelines, the MR has been reviewed by a Technical Writer. -
If the MR changes a component's API, integration MR(s) have been opened in the following projects to ensure that the @gitlab/ui
package can be upgraded quickly after the changes are released:-
GitLab: mr_url -
Customers Portal: mr_url -
Status Page: mr_url
-
-
Added the ~"component:*"
label(s) if applicable.
Security
If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:
-
Label as security and @ mention @gitlab-com/gl-security/appsec
-
Security reports checked/validated by a reviewer from the AppSec team
Closes #1020 (closed)
Edited by Ezekiel Kigbo