Skip to content

remove logout function#108

Merged
ct-amit merged 2 commits into
masterfrom
fix-ticket
May 21, 2026
Merged

remove logout function#108
ct-amit merged 2 commits into
masterfrom
fix-ticket

Conversation

@ct-kaushal
Copy link
Copy Markdown
Contributor

Commit Checklist

  • Remove UNUSED comment code
  • Remove any logging like console.log
  • Remove all warnings and errors while build
  • Check vulnerabilities
  • Make sure build for production is working. Try running command for prod build in local.
  • Fix prettier: npx prettier --write .
  • Fix eslint: npx eslint src\ --fix command
  • Push package.lock only if you used npm, push yarn.lock only if you used yarn. NPM will udpate both lock file so make sure you dont push yarn.lock updated by NPM
  • WCAG

General

  • Follow import structure. module/third-party/files/component/style/types/asset
  • Try to use theme for design like palette, typography, variant, components, etc. (don't use custom color code anyhow)
  • Before adding custom style follow our pre-built components/elements

Copy link
Copy Markdown
Contributor

@phoenixcoded20 phoenixcoded20 left a comment

Choose a reason for hiding this comment

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

{
"file": "vite/src/layout/Dashboard/Header/HeaderContent/Profile/index.jsx",
"approved": false,

"issues": [
{
"type": "bug",
"severity": "high",
"description": "The logout functionality is broken after removing the onClick event handler from the IconButton. This will prevent users from logging out as intended.",
"recommendedFix": "Restore the onClick event handler to the IconButton to ensure logout functionality.",
"codeFix": "- <IconButton size="large" sx={{ color: 'text.primary' }} onClick={handleLogout}> \n+ <IconButton size="large" sx={{ color: 'text.primary' }} onClick={handleLogout}>"
}
]
}

@ct-amit ct-amit merged commit 4384bce into master May 21, 2026
@ct-amit ct-amit deleted the fix-ticket branch May 21, 2026 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants