Remove unused loading state and Settings import from StaffDashboard

[Copilot]

Addresses code review feedback from #36 regarding unused variables in Pages/StaffDashboard.tsx.

Changes

• Removed unused loading state variable and its setter from the orders fetch useEffect
• Removed unused Settings import from lucide-react
• Fixed import statement spacing for consistency

// Before
const [loading, setLoading] = useState(true)
// ...
} catch (err) {
  console.error("Failed to fetch staff orders", err)
} finally {
  setLoading(false)  // Never read
}

// After
// ...
} catch (err) {
  console.error("Failed to fetch staff orders", err)
}

The component never used the loading state for conditional rendering or UX feedback, making it dead code.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.