Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove unused session-fix-confirmation component #250

Merged

Conversation

Arif-Khalid
Copy link
Contributor

Summary:

Fixes part of #72

Type of change:

  • 🎨 Code Refactoring

Changes Made:

  • Removed session-fix-confirmation component. This component was used to confirm creating a new repo for users which is a functionality from CATcher not used in WATcher

Proposed Commit Message:

Remove unused session-fix-confirmation component in WATcher.

This is a component only used in CATcher.

Unnecessary components contributes to code clutter and hurts understandability
of code base.

Let's remove this unused component from WATcher to increase code cleanliness.

Checklist:

  • I have tested my changes thoroughly.
  • I have created tests for any new code files created in this PR or provided a link to a issue/PR that addresses this.
  • I have added or modified code comments to improve code readability where necessary.
  • I have updated the project's documentation as necessary.

@Arif-Khalid Arif-Khalid self-assigned this Feb 8, 2024
@Arif-Khalid Arif-Khalid changed the title Remove unused csession-fix-confirmation component Remove unused session-fix-confirmation component Feb 8, 2024
Copy link
Contributor

@cheehongw cheehongw left a comment

Choose a reason for hiding this comment

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

LGTM

@Arif-Khalid Arif-Khalid merged commit 09ef51d into CATcher-org:main Feb 14, 2024
3 checks passed
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.

2 participants