jfelipeforero / email-client-app

Greetings, I'm Felipe! 👋

I'm a Software Engineer Intern currently working for Snappr

• 🐝 I have worked on several side projects using technologies such as Python, Go ,and Typescript
• 📚 I’m currently learning functional programming with Clojure
• 🌱 I'm looking to get more familiar with AWS technologies

---

Skills for Work

Skills for Side Projects

Currently Getting Familiar

---

Last Edited on: 26/03/2024

email-client-app's Issues

Don't use vite like server static in production

Description

Currently the client is using vite to serve static files in production, vite or vite preview should be used only in development to visualize the application according to the documentation.

The performance of a client can be affected when using a vite or vite preview because they are not optimized to serve medium to high traffic in production.

Steps to reproduce

1. Open the scripts in the package.json
2. The instruction to run the environment of production by standard javascript node is npm run start and this command execute npm run test && npm run dev to finally execute the vite command

Expected vs Actual Behavior

• Expected: The files generated in the build must be delivered by an optimized server.
• Actual: The files generated in the build are delivered by vite previewer.

Possible solution

Configure an express server on the client to serve static files in production

Failed 4 tests for add adition test

Description
Currently fails 4 tests in the backend part in the test auth, test email, test model, test register, test get emails.

Steps to reproduce
1 run the command pip freeze -r > requirements.txt
2 run the command pytest

Expected vs. actual behavior

Expected: Pass all tests
Current: Does not pass 4 tests, only passes 12 tests, one was added to the registry as a suggestion.

Solution

test_register_user: a test was added to the registry so that when the password is empty it does not pass

test_get_all_users: the way in which the data were being tested was changed because they were in a different order and that is why they did not pass.

4 tests remained to be solved
test_registe_user
test_get_mails_user
test_send_email
test_should_get_two_keys this one has a strange problem it seems to have a hidden cache because in the test it does not want to receive changes.