Skip to content
New issue

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

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

Already on GitHub? # to your account

Remove totalSpend field #99

Merged
merged 2 commits into from
Jul 15, 2020
Merged

Conversation

kaplanelad
Copy link
Contributor

What type of PR is this?

/Feature
What this PR does / why we need it:
This column is confusing and it not needed.

Which issue(s) this PR fixes (if exists):

Fixes #93

Special notes for your reviewer:
I haven't removed the logic from the UI.
I'll do it once the new UI will be ready

@kaplanelad kaplanelad added the kind/new-feature New feature label Jul 15, 2020
@kaplanelad kaplanelad self-assigned this Jul 15, 2020
@kaplanelad kaplanelad requested a review from cregev July 15, 2020 13:58
@cregev cregev added the lgtm label Jul 15, 2020
@kaplanelad kaplanelad merged commit cfca773 into master Jul 15, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove the column of Total Spent Price for all resources
2 participants