cuttle icon indicating copy to clipboard operation
cuttle copied to clipboard

Converted the route as mentioned

Open ar1701 opened this issue 1 year ago • 1 comments

Issue number- #563

Relevant #563

  • Resolves #563 : Converted the Route

Please check the following

  • [x] Do the tests still pass? (see Run the Tests)
  • [x] Is the code formatted properly? (see Linting (Formatting))
  • For New Features:
    • [x] Have tests been added to cover any new features or fixes?
    • [x] Has the documentation been updated accordingly?

Please describe additional details for testing this change

ar1701 avatar Oct 02 '24 08:10 ar1701

@ar1701 I love AI and use it all the time! So helpful to learn and it can offer a lot of insight (and power, haha).

That said, it’s incredibly important to double check what it’s telling you and also what it’s doing. When learning, I recommend asking it what you need to do, and then making the changes yourself until you get more comfortable.

seriouslysean avatar Oct 02 '24 12:10 seriouslysean

@ar1701 just so you know for next time, you don't need to open a new PR to make changes to an existing one. You can simply commit and push the changes to your branch and the PR will be updated to reflect those changes.

That's because a pull request is just a UI for merging one branch into another. So whenever we approve and merge a PR, it merges the current latest state of the feature branch and merges that into the base branch (main).

For now, I'm closing this as a duplicate of #1100 since that has the latest changes

itsalaidbacklife avatar Oct 12 '24 17:10 itsalaidbacklife