Skip to content

Added my project#167

Open
Ledana wants to merge 3 commits into
the-csharp-academy:mainfrom
Ledana:main
Open

Added my project#167
Ledana wants to merge 3 commits into
the-csharp-academy:mainfrom
Ledana:main

Conversation

@Ledana
Copy link
Copy Markdown

@Ledana Ledana commented May 6, 2026

I followed the tutorial of drinksinfo. i changed console ui to console.spectre and restclient to htttpclient. i created the favourites functionality where the user can see list and delete favourites. when the user is looking at a drink details if that drink is not in favourites database the user is asked if they wish to add the drink to it. i used ef with sql server to store the favourites. i tried to add the view count but first my idea was to only add it in the favourites drinks and the user could see how many times was a drink viewed but it didnt make sense to only have it for the favourites, and to have it for all the drinks first i needed to get all the drinks and save them in my db so i could save the view count, but i couldnt get all drinks couse the free version of the api didnt have a link for all drinks, maybe im wrong, please tell me if there is a simple way to add the count property for all drinks of all categories.
the image part, well the user can click in the drinkthumb link when they see the drink details, in console i dont think there is a way to put images and if that works with azure im still waiting for my credit card couse azure doesnt take my debit one. ive never worked with azure yet.
i know the code can be still cleaner and more organised, but im still getting into the habit of seperating concerns between methods, classes and parent folders as well.
as always, please be brutally honest!

Copy link
Copy Markdown
Collaborator

@TheCSharpAcademy TheCSharpAcademy left a comment

Choose a reason for hiding this comment

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

@Ledana Thanks for submitting! At the moment the project doesn't build, I can see these errors:

Image

Could you please fix it so we can start the review?

@Ledana
Copy link
Copy Markdown
Author

Ledana commented May 15, 2026

jeez, how could i forget that. fixed! :)

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