Skip to content

Conversation

@Kasssiiii
Copy link

@Kasssiiii Kasssiiii commented Aug 17, 2025

Copy link

@JennieDalgren JennieDalgren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job with the project.

However, there is a bug. When posting a thought there is an old thought getting stored in the list. See my screenshots:

  1. Im typing my thought...
Screenshot 2025-08-25 at 17 00 24
  1. When I press send, this happens.
Screenshot 2025-08-25 at 17 00 29

FIx this when you have time and ping me when it's done.

@T-Thiry
Copy link

T-Thiry commented Oct 5, 2025

Good job, Kasia! You’ve got the core functionality in place and the app is coming together nicely ! 🙌🌟 Nice touch with the focus state on the text field, makes it clear where the user is typing. I really like that the app greets the user by name, it feels personal and welcoming!

Thing to improve from looking visiting the deployed website:

  • When I submitted a message ("Sun is shining in GBG"), the post displayed “I am happy too” instead. I only saw my own message after logging out, maybe something is off with how the message is saved or displayed for logged-in users.
  • You might want to adjust the container width so it doesn’t take up quite as much space, it’ll help the layout feel more balanced.
  • The second heart icon could be aligned with the rest of the elements inside the post to make the layout a bit cleaner.
  • On smaller screens, part of the form at the top disappears, so adding some responsive adjustments there could improve usability on mobile.

Things to improve from looking at the code:

  • Split up the code into smaller files for better separation of concerns.
  • Move the schemas (User and Thought) into a /models folder.
  • Put the authentication middleware into its own /middleware/auth.js file.
  • Move the routes for users and thoughts into separate files (like /routes/userRoutes.js and /routes/thoughtRoutes.js).
  • Fixing typos such as require: true (should be required)

@JennieDalgren
Copy link

Still the same weird bug when trying to add a new thought.

Check this short screen recording

https://supercut.ai/share/technigo/XqZVaXZIG0qyJuDW8YZjYi

@HIPPIEKICK
Copy link
Contributor

ping

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants