Skip to content

Javier Velilla
jvelilla

Organizations

@eiffelhub @EiffelWebFramework
jvelilla commented on pull request EiffelWebFramework/Redwood#2
@jvelilla

Missing description

jvelilla commented on pull request EiffelWebFramework/Redwood#2
@jvelilla

General comment: try to add style to the Readme file, because at the moment is not easy to read. Markdown syntax: https://daringfireball.net/projec…

jvelilla commented on pull request EiffelSoftware/EiffelStudio#5
@jvelilla

Yes it was committed. On Thu, Jan 28, 2016 at 7:58 AM, Emmanuel Stapf notifications@github.com wrote: Was this committed at the end? If so we shou…

jvelilla opened pull request blank23/firebase4Eiffel#1
@jvelilla
Firebase rev
3 commits with 81 additions and 59 deletions
@jvelilla
  • @jvelilla 0ba9abc
    Added template TEST_REST_API
jvelilla deleted branch jv_review at jvelilla/imap_eiffel_library
@jvelilla
@jvelilla
  • @jvelilla 2f95c66
    Removed unneeded template
jvelilla commented on pull request EiffelWebFramework/ROC#48
@jvelilla

done

jvelilla commented on pull request EiffelWebFramework/ROC#48
@jvelilla

Yes, now it's part of user.sql

jvelilla commented on pull request EiffelWebFramework/ROC#48
@jvelilla

Yes and it's not used. I will remove it.

@jvelilla
  • @jvelilla f244e86
    Updated user.sql added tabled auth_temp_users.
@jvelilla
  • @jvelilla 0cf6e59
    Updated templates and fixed typos.
jvelilla commented on pull request EiffelWebFramework/ROC#48
@jvelilla

In fact I think it's not needed at all, because the interface does not check the user has_id in the postcondition.

jvelilla commented on pull request EiffelWebFramework/ROC#48
@jvelilla

Removed

jvelilla commented on pull request EiffelWebFramework/ROC#48
@jvelilla

Ok good idea

jvelilla commented on pull request EiffelWebFramework/ROC#48
@jvelilla

I agree, but at the moment CMS_TEMP_USER is not part of the core, so we can move it to the core to reuse CMS_USER_API

jvelilla commented on pull request EiffelWebFramework/ROC#48
@jvelilla

I don't think so, because at the moment CMS_TEMP_USER is not part of the core.

jvelilla commented on pull request EiffelWebFramework/ROC#48
@jvelilla

fixed

jvelilla commented on pull request EiffelWebFramework/ROC#48
@jvelilla

not at the moment, because user profile is not stored for now. And we have to review CMS_USER, because it also have items, and it seems they repres…

jvelilla commented on pull request EiffelWebFramework/ROC#48
@jvelilla

done

jvelilla commented on pull request EiffelWebFramework/ROC#48
@jvelilla

done

jvelilla commented on pull request EiffelWebFramework/ROC#48
@jvelilla

done

jvelilla commented on pull request EiffelWebFramework/ROC#48
@jvelilla

done

jvelilla commented on pull request EiffelWebFramework/ROC#48
@jvelilla

Updated to : User application

jvelilla commented on pull request EiffelWebFramework/ROC#48
@jvelilla

fixed

jvelilla commented on pull request EiffelWebFramework/ROC#48
@jvelilla

fixed type

Something went wrong with that request. Please try again.