search

cjstyles6 / Currency-Converter

0 stars 2 forks source link

Refactor codebase for readability #3

Open cjstyles6 opened 5 months ago

cjstyles6 commented 5 months ago

The current codebase works as intended but could be improved for maintainability and extensibility.Like using custom hooks

Sky-De commented 4 months ago

@cjstyles6 assignment please

cjstyles6 commented 4 months ago

sure,please make sure the application still runs properly when you refactor it

Sky-De commented 4 months ago

@cjstyles6 It's not working properly already. I turned off the fetch delay to capture a short video for you, the first currency switch isn't functioning and stays the same take a look at this:

chrome-capture-2024-1-11 (2)

Are you willing to fix that, or should I leave it as is?

cjstyles6 commented 4 months ago

someone already fixed it,you can go ahead

Sky-De commented 4 months ago

@cjstyles6 already fixed it,you can go ahead

I was busy for a couple of days. Thank you for letting me know about the fixed issue. I will take care of it tonight.

Sky-De commented 4 months ago

someone already fixed it,you can go ahead

I have checked your app again and unfortunately, the issue that I previously reported still exists, let me know as soon as this issue addressed then I will be able to refactor code thanks

cjstyles6 commented 4 months ago

Just fix the issue if you can

Sky-De commented 1 month ago

@cjstyles6 hi, please close this issue, thanks