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

Ticket out existing TODOs #134

Open
ezra-sg opened this issue Sep 9, 2022 · 0 comments
Open

Ticket out existing TODOs #134

ezra-sg opened this issue Sep 9, 2022 · 0 comments
Labels
🔧 Tech Debt Refactoring, library upgrades, documentation

Comments

@ezra-sg
Copy link
Contributor

ezra-sg commented Sep 9, 2022

Overview

There are 11 TODOs (and an unknown number of commented lines of code / TODO-like comments) in the application right now, some of which seem rather important. These TODOs and comments are effectively lost. They should either be added to the backlog in the form of Issues or removed.

Acceptance criteria

  • all TODOs and TODO-like comments should be removed from the codebase and placed in a GitHib Issue

Technical considerations

  • search this repo's GitHub issues to see if any describes the same thing as the TODO
    • if not, create a new issue to address the TODO
  • replace the TODO in the code with a comment simply linking to the issue, eg.
    // TODO https://github.com/telosnetwork/teloscan/issues/134

Tasks

@ezra-sg ezra-sg added the 🔧 Tech Debt Refactoring, library upgrades, documentation label Sep 9, 2022
@ezra-sg ezra-sg added 🔬 Needs Refinement Further information is required and removed 🔬 Needs Refinement Further information is required labels Oct 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔧 Tech Debt Refactoring, library upgrades, documentation
Projects
None yet
Development

No branches or pull requests

1 participant