Skip to content

Conversation

@mahineelam
Copy link
Contributor

Related Ticket: {link related ticket here}

Description of Changes

{Update with description of changes made in this pull request}

Notes & Questions About Changes

{Add additonal notes and outstanding questions here related to changes in this pull request}

Validation / Testing

{Update with info on what can be manually validated in the Deploy Preview link for example "Validate style updates to selection modal do NOT affect cards"}

@netlify
Copy link

netlify bot commented Sep 10, 2025

Deploy Preview for veda-ui failed.

Name Link
🔨 Latest commit 5a1b8cb
🔍 Latest deploy log https://app.netlify.com/projects/veda-ui/deploys/68c1a18e462fbf000847af48

@mahineelam mahineelam changed the title Trop pressure feat(tropess): add dynamic pressure level selection Sep 10, 2025
@AliceR
Copy link
Member

AliceR commented Sep 11, 2025

Hi @mahineelam , thanks for opening a Pull Request in veda-ui! To help us review it properly, could you update the description with some details on what you’re aiming to do? Our PR template can be a good guide if that helps.

@ifsimicoded
Copy link
Contributor

Hi @mahineelam, it looks like you're adding a test dataset. In general, datasets should be managed by instances of veda-ui. As @AliceR mentioned, curious what your intention for this change is. As a matter of repo hygiene, I'll be closing this PR if we don't hear back by end of week (Nov 7th).

@ifsimicoded ifsimicoded added invalid This doesn't seem right response required If comments are not addressed in a reasonable amount of time, the PR will be closed and removed invalid This doesn't seem right labels Nov 3, 2025
@vgeorge vgeorge closed this Nov 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

response required If comments are not addressed in a reasonable amount of time, the PR will be closed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants