Skip to content
This repository has been archived by the owner on Mar 18, 2021. It is now read-only.

COM-20065: Implement card view for bundle list #144

Conversation

ciastektk
Copy link
Collaborator

@ciastektk ciastektk commented Nov 9, 2018

@ciastektk ciastektk force-pushed the COM-20065-implement-card-view-for-bundle-list branch from 46b57e8 to c86dcba Compare November 9, 2018 12:42
Copy link
Collaborator

@SylvainGuittard SylvainGuittard left a comment

Choose a reason for hiding this comment

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

Thanks @ciastektk for this improvement

@@ -0,0 +1,54 @@
<div class="bundle-card-line-container{% if extra.page is defined %} page-{{ extra.page }}{% endif %}">
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't think this file should be in this PR ;)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Nikita wants to keep old template in repository

Copy link
Collaborator

Choose a reason for hiding this comment

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

This is why we have git. So we can restore the previous version of the file if we need to.
There is no need to keep an unused file in the repo.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok, I removed this template

@ciastektk ciastektk force-pushed the COM-20065-implement-card-view-for-bundle-list branch 2 times, most recently from 7c52c0f to c28f90b Compare November 14, 2018 11:59
@ciastektk ciastektk force-pushed the COM-20065-implement-card-view-for-bundle-list branch from c28f90b to f834612 Compare November 15, 2018 13:00
Copy link
Collaborator

@SylvainGuittard SylvainGuittard left a comment

Choose a reason for hiding this comment

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

Approved! Thanks @ciastektk
(if you can fix my last comment, that would be great)

@damianz5 damianz5 self-assigned this Nov 15, 2018
Co-Authored-By: ciastektk <tomasz.kryszan@ez.no>
@damianz5 damianz5 merged commit 31bc95a into ezsystems:master Nov 15, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

None yet

3 participants