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

add recently updated acopios by default #85

Merged
merged 3 commits into from
Sep 24, 2017

Conversation

ldelriof
Copy link
Contributor

queda pendiente asegurar que la fechaDeActualizacion de acopios se actualize cuando un producto es añadido de parte del API

Copy link
Contributor

@jeduan jeduan left a comment

Choose a reason for hiding this comment

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

Este commit rompe /list

@@ -22,13 +22,13 @@ class AcopioCard extends Component {

const {
nombre,
products,
Copy link
Contributor

Choose a reason for hiding this comment

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

Esto no rompe las otras pantallas que usan este componente?

@@ -10,7 +10,7 @@ const Layout = ({ children, history }) => {
<div>
<AppBar
style={{position: 'fixed', top: '0', backgroundColor: '#191E1B'}}
title={appName}
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

const { positionUnknown, positionUnavailable } = this.props
console.log(acopios)
Copy link
Contributor

Choose a reason for hiding this comment

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

console.log extra


if (positionUnknown) {
return <span>Permite tu ubicación</span>
return (
this.getAcopiosList(title, isLoading, acopios)
Copy link
Contributor

Choose a reason for hiding this comment

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

esto usa title antes de que se defina abajo

@jeduan jeduan merged commit 30a470f into develop Sep 24, 2017
@jeduan jeduan deleted the feature/recently_updated_acopios branch September 24, 2017 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants