ajoliat23 / blockBook

0 stars 0 forks source link

Add twitter clone webpage and switch .js to .tsx #24

Closed Zachtmac1 closed 1 year ago

Zachtmac1 commented 1 year ago

Purpose: The main purpose of this pull request is to add the twitter clone code to main. It has been almost entirely built out including the CSS styling. The only portion that still needs to be addressed is the source of the information for the actual posts. The database connection needs to be fleshed out, and starter code is included for when that issue is dealt with.

Other tasks: Additionally, the twitter clone was originally written in JavaScript, so time was spent converting those files to typescript and addressing the resulting errors. The nft search related files were also switched over to typescript during this process.

Debugging: Much time was also spent making sure that there would be no issues with the base branch, and to do this I created another branch of main and merged this branch with that one and fixed the resulting merge conflicts. Because of this, the transition into main should be seamless. The import issue with materials-ui was also figured out, as it turns out the import was deprecated and a new import name was supposed to be used.

Drawbacks: There were a few drawbacks to the way this was approached, namely the decision to write it with JavaScript and then switch to Typescript vs just starting out with Typescript.