Skip to content

Project-api hand in#51

Open
RPaemurd wants to merge 11 commits intoTechnigo:masterfrom
RPaemurd:master
Open

Project-api hand in#51
RPaemurd wants to merge 11 commits intoTechnigo:masterfrom
RPaemurd:master

Conversation

@RPaemurd
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@HIPPIEKICK HIPPIEKICK left a comment

Choose a reason for hiding this comment

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

Some missing requirements:

  • Possibility to edit and delete in frontend
  • Thoughts are not saved (when refreshing the frontend, my newly created thought is gone)

@RPaemurd
Copy link
Copy Markdown
Author

RPaemurd commented Apr 1, 2026

@HIPPIEKICK The thoughts were saved but they were puplished at the and of the site, not at the beginning, so you probably just didnt see them. My bad. They are now at the top of the thoughts list.
changed
from: const thoughts = await Thought.find()

to: const thoughts = await Thought.find().sort({ createdAt: -1 })

I had the PUT and DELETE already in backend, lacked in frontend though. I will fix!

@RPaemurd
Copy link
Copy Markdown
Author

RPaemurd commented Apr 2, 2026

@HIPPIEKICK I think I have fixed everything, let me know if its not working :)

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.

2 participants