Move Heading component to staff feature flag (#4915)

* Move Heading component to staff feature flag * Create dull-dolphins-bake.md
primer · Sep 4, 2024 · 69922d1 · 69922d1
1 parent 7a24a01
commit 69922d1
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 7 deletions.
diff --git a/.changeset/dull-dolphins-bake.md b/.changeset/dull-dolphins-bake.md
@@ -0,0 +1,5 @@
+---
+"@primer/react": patch
+---
+
+Move Heading component to staff feature flag
diff --git a/e2e/components/Heading.test.ts b/e2e/components/Heading.test.ts
@@ -28,7 +28,7 @@ test.describe('Heading', () => {
           id: story.id,
           globals: {
             featureFlags: {
-              primer_react_css_modules_team: true,
+              primer_react_css_modules_staff: true,
             },
           },
         })
@@ -51,7 +51,7 @@ test.describe('Heading', () => {
           id: story.id,
           globals: {
             featureFlags: {
-              primer_react_css_modules_team: true,
+              primer_react_css_modules_staff: true,
             },
           },
         })

diff --git a/packages/react/src/Heading/Heading.tsx b/packages/react/src/Heading/Heading.tsx
@@ -37,7 +37,7 @@ const StyledHeading = styled.h2<StyledHeadingProps>`
 `
 
 const Heading = forwardRef(({as: Component = 'h2', className, variant, ...props}, forwardedRef) => {
-  const enabled = useFeatureFlag('primer_react_css_modules_team')
+  const enabled = useFeatureFlag('primer_react_css_modules_staff')
   const innerRef = React.useRef<HTMLHeadingElement>(null)
   useRefObjectAsForwardedRef(forwardedRef, innerRef)
 

diff --git a/packages/react/src/Heading/__tests__/Heading.test.tsx b/packages/react/src/Heading/__tests__/Heading.test.tsx
@@ -142,12 +142,12 @@ describe('Heading', () => {
     ).toHaveStyleRule('font-style', 'italic')
   })
 
-  describe('with primer_react_css_modules_team enabled', () => {
+  describe('with primer_react_css_modules_staff enabled', () => {
     it('should only include css modules class', () => {
       HTMLRender(
         <FeatureFlags
           flags={{
-            primer_react_css_modules_team: true,
+            primer_react_css_modules_staff: true,
           }}
         >
           <Heading>test</Heading>
@@ -163,7 +163,7 @@ describe('Heading', () => {
       const {container} = HTMLRender(
         <FeatureFlags
           flags={{
-            primer_react_css_modules_team: true,
+            primer_react_css_modules_staff: true,
           }}
         >
           <Heading className="test">test</Heading>
@@ -176,7 +176,7 @@ describe('Heading', () => {
       HTMLRender(
         <FeatureFlags
           flags={{
-            primer_react_css_modules_team: true,
+            primer_react_css_modules_staff: true,
           }}
         >
           <Heading