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.

Sign up for GitHub

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

Jump to bottom

Manager Module layouts broken with recent architecture changes #8

Open
MichaelJ2324 opened this issue May 19, 2015 · 4 comments
Open

Manager Module layouts broken with recent architecture changes #8

MichaelJ2324 opened this issue May 19, 2015 · 4 comments
Assignees
@UTAlan
Labels
Backbone.js bug In Progress
Milestone
Beta Release

Comments

@MichaelJ2324
Copy link
Owner

No description provided.

@MichaelJ2324 MichaelJ2324 added this to the Beta Release milestone May 19, 2015
@UTAlan UTAlan self-assigned this May 22, 2015
@UTAlan
Copy link
Collaborator

UTAlan commented May 22, 2015

Got rid of the this.templates errors when loading the Manager module. Trying to track down the root of these now:

GET http://unbox.localhost/api/undefined 405 (Method Not Allowed)
GET http://unbox.localhost/api/undefined 405 (Method Not Allowed)

@MichaelJ2324
Copy link
Owner Author

Check out Bootstrap in UNBOXAPI.Views.Layout.....

Essentially, when a Layout is loaded, there is a built in option (see module.php config file, bootstrap property) to bootstrap data for the entire layout. When I initially built it, it made sense for some things, but I haven't worked it back in and the models will atempt to pull data that doesn't exist. Either remove the definition in module.php for bootstrap property or.....yeah just do that, I want bootstrap in there for future stuff.

@MichaelJ2324
Copy link
Owner Author

Added an issue for this for further tracking #13

UTAlan added a commit to UTAlan/UNBOX_API that referenced this issue May 22, 2015
@UTAlan
Fixed the Manager Layout errors, per Issue MichaelJ2324#8
6bebd00
@UTAlan UTAlan mentioned this issue May 22, 2015
Merged
@UTAlan UTAlan removed their assignment May 22, 2015
@UTAlan
Copy link
Collaborator

UTAlan commented May 22, 2015

Awesome. Got that taken care of, pull request created.

MichaelJ2324 added a commit that referenced this issue May 22, 2015
@MichaelJ2324
Merge pull request #18 from UTAlan/master
c23c471 
Fixed the Manager Layout errors, per Issue #8. Issue #8 still open
MichaelJ2324 added a commit that referenced this issue May 22, 2015
@MichaelJ2324
Backbone View Changes
5a0a868 
Backbone view changes. Manager and Tester should have some fixes
applied. Issue #8 and Issue #7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@UTAlan UTAlan

Labels
Backbone.js bug In Progress
Projects
None yet
Milestone
Beta Release
Development

No branches or pull requests
2 participants
@UTAlan @MichaelJ2324