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.

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

SL/TP clean up #155

Merged
merged 5 commits into from
May 3, 2024
Merged

SL/TP clean up #155

merged 5 commits into from
May 3, 2024

Conversation

mike-dydx
Copy link
Collaborator

@mike-dydx mike-dydx commented May 3, 2024

Links (dYdX Internal Use Only)

Linear Ticket: MOB-447 : Test on small devices


Description

  • removed comments and prints
  • display oracle price as dollar formatted
  • only call onEdited callback during user edits
  • handle the case when user makes no changes to existing sl/tp such that pending orders is 0
  • change default theme to classic_dark

Before/After Screenshots or Videos

Before After

@mike-dydx mike-dydx requested a review from ruixhuang May 3, 2024 16:13
@mike-dydx mike-dydx marked this pull request as draft May 3, 2024 16:13
@mike-dydx mike-dydx marked this pull request as ready for review May 3, 2024 17:26
@@ -18,7 +18,7 @@ public enum dydxSettingsStoreKey: String, CaseIterable {
public var defaultValue: Any? {
switch self {
case .language: return DataLocalizer.shared?.language
case .v4Theme: return dydxThemeType.dark.rawValue
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

i checked this change with Achal

@mike-dydx mike-dydx requested a review from johnqh May 3, 2024 18:11
@ruixhuang ruixhuang merged commit 629f7d6 into develop May 3, 2024
2 checks passed
@ruixhuang ruixhuang deleted the mm/sl-tp-cleanup branch May 3, 2024 18:30
mike-dydx added a commit that referenced this pull request Aug 20, 2024
* clean up

* default to classic_dark instead of dark

* display oracle price and entry price as dollar formatted values

* handle pendingOrders == 0

* call on edited only when focused
mike-dydx added a commit that referenced this pull request Aug 21, 2024
* clean up

* default to classic_dark instead of dark

* display oracle price and entry price as dollar formatted values

* handle pendingOrders == 0

* call on edited only when focused
mike-dydx added a commit that referenced this pull request Aug 21, 2024
* clean up

* default to classic_dark instead of dark

* display oracle price and entry price as dollar formatted values

* handle pendingOrders == 0

* call on edited only when focused
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants