hakowork/rojo
3
0
Fork 0
mirror of https://github.com/rojo-rbx/rojo.git synced 2026-10-23 05:51:39 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Update button layout order in ConfirmingPage and enhance error handling in change processor

Browse source 
- Changed the layout order of the reject button in the ConfirmingPage to improve UI consistency.
- Added error handling for file operations in the change processor, logging errors when file removal or writing fails, enhancing robustness and debuggability.
This commit is contained in:
Joe 2026-01-24 05:12:46 -06:00
parent 06d5ff4d80
commit 8650cdbd82
2 changed files with 20 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

2
plugin/src/App/StatusPages/Confirming.lua
View file

@ -95,7 +95,7 @@ function ConfirmingPage:render()
text = "Reject",
style = "Warning",
transparency = self.props.transparency,
layoutOrder = 2,
layoutOrder = 0,
onClick = function()
self:setState({ showingRejectConfirm = true })
end,

22
src/change_processor.rs
View file

@ -200,7 +200,16 @@ impl JobThreadContext {
if let Some(instance) = tree.get_instance(id) {
if let Some(instigating_source) = &instance.metadata().instigating_source {
match instigating_source {
InstigatingSource::Path(path) => fs::remove_file(path).unwrap(),
InstigatingSource::Path(path) => {
if let Err(err) = fs::remove_file(path) {
log::error!(
"Failed to remove file {:?} for instance {:?}: {}",
path,
id,
err
);
}
}
InstigatingSource::ProjectNode { .. } => {
log::warn!(
"Cannot remove instance {:?}, it's from a project file",
@ -244,14 +253,21 @@ impl JobThreadContext {
match instigating_source {
InstigatingSource::Path(path) => {
if let Some(Variant::String(value)) = changed_value {
fs::write(path, value).unwrap();
if let Err(err) = fs::write(path, value) {
log::error!(
"Failed to write Source to {:?} for instance {:?}: {}",
path,
id,
err
);
}
} else {
log::warn!("Cannot change Source to non-string value.");
}
}
InstigatingSource::ProjectNode { .. } => {
log::warn!(
"Cannot remove instance {:?}, it's from a project file",
"Cannot update instance {:?}, it's from a project file",
id
);
}