Skip to content

My finished challenge - #762

Open
lucascorreia-tech wants to merge 1 commit into
the-csharp-academy:mainfrom
lucascorreia-tech:main
Open

My finished challenge#762
lucascorreia-tech wants to merge 1 commit into
the-csharp-academy:mainfrom
lucascorreia-tech:main

Conversation

@lucascorreia-tech

Copy link
Copy Markdown

No description provided.

@github-actions

Copy link
Copy Markdown

🚀 Project Submitted!

Thanks for submitting your project! I will review it soon.

📋 Please check the dashboard to make sure the project is marked as Pending.

⚠️ If it's not marked as pending, submit it manually using the URL for this PR.

@TheCSharpAcademy TheCSharpAcademy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@lucascorreia-tech Thanks for submitting your project. There are a couple of problems that you need to fix before we can review. First there's a bug where the app crashes when it tries to divide by zero:

Image

Additionally your app doesn't let the users choose the operation, which is a requirement of the project:

Image

Let me know if you need help, sou brasileiro tambem. 😁

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