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

Remove Deposit#fund_extra, Deposit#fund_uid and usages (fixes #803) #820

Merged
merged 3 commits into from Apr 10, 2018

Conversation

k1T4eR
Copy link

@k1T4eR k1T4eR commented Apr 7, 2018

No description provided.

Copy link

@yivo yivo left a comment

Choose a reason for hiding this comment

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

I don't understand why you always misses the locales.
image

@@ -18,7 +18,6 @@ <h2 class="panel-title">{{ "funds.deposit_history.title" | t }}</h2>
<tr class="deposit_item" ng-class="{ 'new-row':dhCtrl.newRecord(deposit) }" ng-repeat="deposit in dhCtrl.deposits | orderBy:predicate">
<td><span class="outlined-label">{{ deposit.id }}</span></td>
<td>{{ deposit.created_at | date:'yyyy-MM-dd HH:mm' }}</td>
<td>{{ deposit.fund_extra }} @ {{ deposit.fund_uid }}</td>
Copy link

Choose a reason for hiding this comment

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

@k1T4eR This will not work. You should test it manually.

@@ -7,12 +7,6 @@
</div>
<div class="panel-body">
<dl class="dl-horizontal dl-member">
<dt>Bank</dt>
Copy link

Choose a reason for hiding this comment

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

@k1T4eR Send me the screenshot of deposit form.

Copy link
Author

Choose a reason for hiding this comment

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

image

@mod mod merged commit 012f99a into openware:master Apr 10, 2018
ysv pushed a commit to ysv/peatio that referenced this pull request Apr 13, 2018
…e#803) (openware#820)

* Remove Deposit#fund_extra, Deposit#fund_uid and usages

* Remove fund_ui fund_extra_text from locales.

* Remove 'deposit_history.from' column.
@k1T4eR k1T4eR deleted the #803 branch June 7, 2018 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants