Skip to content

Adicionando Teste Técnico de Desenvolvimento Moveis Simonetti #61

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

genivanms
Copy link

Adicionando os arquivos do teste tecnico de desenvolvimento da Moveis Simonetti.

Copy link
Member

@lucassabreu lucassabreu left a comment

Choose a reason for hiding this comment

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

apagar o node_modules tbm

@genivanms
Copy link
Author

genivanms commented Apr 4, 2025 via email

Comment on lines 30 to 50
echo "Buscando investimento ID: " . $investmentId . "<br>";

$investment = $this->getInvestmentById($investmentId);

if (!$investment) {
echo "Investimento não encontrado!<br>";
echo "URL do banco: " . $_ENV['DATABASE_URL'] . "<br>";
return new JsonResponse(['error' => 'Investment not found'], 404);
}

echo "Investimento encontrado!<br>";
echo "Nome do proprietário: " . $investment->getOwner()->getName() . "<br>";
echo "Valor: " . $investment->getValue() . "<br>";

$investmentDetails = $this->calculateInvestmentDetails($investment);

echo "<pre>";
print_r($investmentDetails);
echo "</pre>";

exit;
Copy link
Member

Choose a reason for hiding this comment

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

@genivanms eu estava começando o review agora, mas vi isso aqui, e não achei uma explicação de como fazer o setup do projeto, apenas da documentação.

quer mexer nesses pontos antes de eu continuar?

Choose a reason for hiding this comment

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

Bom dia, sim irei mexer agora... no caso pela documentação vc não conseguiu testar / fazer funcionar e isso?

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.

3 participants