Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Development 313 simulator update #1228

Merged
merged 2 commits into from
Oct 17, 2023
Merged

Development 313 simulator update #1228

merged 2 commits into from
Oct 17, 2023

Conversation

maayarosama
Copy link
Contributor

Description

  • updated the net profit scale to start by 0.01
  • updated token price after 5 years = 1$ as default
  • updated the ROI field to reflect the right roi value

Changes

Related Issues

#1042

Checklist

  • Tests included
  • Build pass
  • Documentation
  • Code format and docstrings
  • Screenshots/Video attached (needed for UI changes)

@maayarosama maayarosama changed the base branch from development to development_313 October 17, 2023 08:42
@maayarosama maayarosama merged commit fa08988 into development_313 Oct 17, 2023
@maayarosama maayarosama deleted the development_313_simulator_update branch October 17, 2023 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MohamedElmdary MohamedElmdary MohamedElmdary approved these changes

@AhmedHanafy725 AhmedHanafy725 Awaiting requested review from AhmedHanafy725

@zaelgohary zaelgohary Awaiting requested review from zaelgohary

@Mahmoud-Emad Mahmoud-Emad Awaiting requested review from Mahmoud-Emad

@amiraabouhadid amiraabouhadid Awaiting requested review from amiraabouhadid

@Omarabdul3ziz Omarabdul3ziz Awaiting requested review from Omarabdul3ziz

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@maayarosama @MohamedElmdary