couchdb-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From robertkowalski <...@git.apache.org>
Subject [GitHub] couchdb-fauxton pull request: Initial version of an all-purpose lo...
Date Fri, 31 Oct 2014 12:24:28 GMT
Github user robertkowalski commented on the pull request:

    https://github.com/apache/couchdb-fauxton/pull/119#issuecomment-61252853
  
    I played a bit with the code and I feel unsure about the `reinitialize`. At least we should
write a test that documents that behaviour change and I am happy about further discussion.
    
    While playing with the code I came up with other solutions, but so far I am not really
happy with them:
    
     - we could fire a `routeEvent` and in the handler the backbone model for the database
can be changed and views could listen to the `change` event of the model and then rerender.
This is alot of manual work and also introduces a chain of events
    
     - Just to get a feeling how it looked like I also tried to take most of the code in the
initialize method and put it into a `initViews` method and then call it where it is needed
but when I looked at it, it did not was the way I wanted it to be


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message