Skip to content

Conversation

@mariana-furyk
Copy link
Contributor

πŸ“ Description

Update MLrun UI repo from React vs 18 to 19


πŸ› οΈ Changes Made

  • Upgrade React 18 to 19
  • Refactor code (use new hooks, deprecate old hooks)y
  • Update 3rd party packages

βœ… Checklist

  • I have given the PR a well-structured title describing the domain and the specific change that was made
  • I tested the changes in the browser (locally or via preview build)
  • I confirmed that existing tests pass
  • I checked that this change doesn’t introduce new console warnings or lint / formatting errors
  • I updated the relevant Jira ticket with the appropriate details and status

πŸ”— References


🚨 Potentially Breaking Changes

  • Yes

Includes DRC change

  • Yes

@mariana-furyk mariana-furyk marked this pull request as draft November 20, 2025 18:59
},
[searchRef]
)
// === ΠžΠ±Ρ‡ΠΈΡΠ»Π΅Π½Π½Ρ ΡˆΠΈΡ€ΠΈΠ½ΠΈ Ρ–Π½ΠΏΡƒΡ‚Π° після монтування ===
Copy link
Collaborator

Choose a reason for hiding this comment

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

pls remove comnent

}) => {
const [searchValue, setSearchValue] = useState(value ?? '')
const [label, setLabel] = useState('')
// const [label, setLabel] = useState('')
Copy link
Collaborator

Choose a reason for hiding this comment

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

if we don't need it, pls remove

}

onChange(value)
// === Основна Π»ΠΎΠ³Ρ–ΠΊΠ° оновлСння значСння ===
Copy link
Collaborator

Choose a reason for hiding this comment

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

pls remove the comment


const matchOnClick = item => {
setLabel('')
// === ΠšΠ»Ρ–ΠΊ Π½Π° Π΅Π»Π΅ΠΌΠ΅Π½Ρ‚ Π·Ρ– списку ===
Copy link
Collaborator

Choose a reason for hiding this comment

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

pls remove the comment

}

const handleSearchIconClick = event => {
// === ΠšΠ»Ρ–ΠΊ ΠΏΠΎ Ρ–ΠΊΠΎΠ½Ρ†Ρ– ΠΏΠΎΡˆΡƒΠΊΡƒ ===
Copy link
Collaborator

Choose a reason for hiding this comment

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

pls remove the comment

dataInputState.storePathType
)
}))
//TODO
Copy link
Collaborator

Choose a reason for hiding this comment

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

pls add more specific todo

setFunctions([])
})

//TODO:
Copy link
Collaborator

Choose a reason for hiding this comment

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

pls add more specific todo


const formRef = React.useRef(
createForm({
// const formRef = React.useRef(
Copy link
Collaborator

Choose a reason for hiding this comment

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

pls remove

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.

2 participants