Skip to content
This repository has been archived by the owner on Mar 26, 2022. It is now read-only.

Clean Up App, Add Encryption Error Handling, and Add Tests #72

Open
torch2424 opened this issue Dec 4, 2016 · 2 comments
Open

Clean Up App, Add Encryption Error Handling, and Add Tests #72

torch2424 opened this issue Dec 4, 2016 · 2 comments

Comments

@torch2424
Copy link
Member

torch2424 commented Dec 4, 2016

As the title states, due to the professor moving up deadlines, and springing up random 13 page reports, and trust me, I just re-read the initial project documents to make sure I wasn't crazy, the front end and back end have a good amount of Tech Debt. The app started just fine, and had a lot of good things going on, but now there is alot of code just thrown around, and the schemas are REALLY bad. Like 5 objects deep to get a PGP key bad...

Also, the backend AND the frontend are half handling users by the mongo ids, and the other half by their facebook ids.

And not to mention the 20+ step process it will be to take upgrade Ionic 2 from beta 11 to RC4 or whatever current ionic version...

May these cynical gifs express the feelings of the tech debt in this project:

Tech Debt

More Tech Debt

Even More Tech Debt

Oh Wait, Did I forget to mention the tech debt in this project?

@kuminin
Copy link
Contributor

kuminin commented Dec 4, 2016

I had to change to User_Ids half way through the project so we could implement the group chat >_<

Also the storage for messages and the pgp keys in the db were quite janky LOL I might fix that in the future :P

@torch2424
Copy link
Member Author

@kuminin Hahaha It's cool dude :D We live and learn :D

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants