Skip to content

fix(ix-time-picker): deploy PR 2508 changes in scope of IX-3896#226

Draft
alexkaduk wants to merge 4 commits intomainfrom
feature/IX-3896-min-max-time
Draft

fix(ix-time-picker): deploy PR 2508 changes in scope of IX-3896#226
alexkaduk wants to merge 4 commits intomainfrom
feature/IX-3896-min-max-time

Conversation

@alexkaduk
Copy link
Copy Markdown
Collaborator

💡 What is the current behavior?

GitHub Issue Number: #

🆕 What is the new behavior?

👨‍💻 Help & support

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 21, 2026

Deploy Preview for industrial-experience ready!

Name Link
🔨 Latest commit eaeda19
🔍 Latest deploy log https://app.netlify.com/projects/industrial-experience/deploys/69e9ec180d4b48000880ec0d
😎 Deploy Preview https://deploy-preview-226--industrial-experience.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds a new environment configuration file and removes a blank line in the home page. The feedback suggests using CI/CD environment variables instead of hardcoding the pull request number to improve maintainability.

Comment thread .env.pullrequest
DOCS_BRANCH='main'
DOCS_BRANCH_TYPE='pull request'

DOCS_PR_NUMBER='2508'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Hardcoding the PR number ('2508') in a configuration file makes it specific to this pull request and creates a maintenance burden. Consider using CI/CD environment variables to dynamically provide this value during the build process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant