systemapic / systemapic.js

Systemapic Javascript API
https://systemapic.com
0 stars 3 forks source link

Project Full Screen User List still buggy #155

Closed jorgenevil closed 8 years ago

jorgenevil commented 8 years ago

There are several bugs with this:

As you see in the console, the error Cannot read property 'getFullName' of undefined appears on this line: https://github.com/systemapic/systemapic.js/blob/1cc6fe9066c9f221032647f30744daf51c534c8d/js/src/chrome/projects/chrome.projects.js#L993

First time opening full screen: screen shot 2016-02-24 at 3 47 06 pm

Second time opening full screen: screen shot 2016-02-24 at 3 48 47 pm

jorgenevil commented 8 years ago

This is what it looks like: full-pane

knutole commented 8 years ago

Wow, great BUG FEEDBACK bro! :)

knutole commented 8 years ago

@IgorZiegler You can download http://www.cockos.com/licecap/ to create GIF's for issues :)

IgorZiegler commented 8 years ago

@knutole @jorgenevil Could you shared this project with me? I have a problem with reproduce it.

jorgenevil commented 8 years ago

@IgorZiegler You should already be an editor of this project, on beta server: https://beta.systemapic.com/jorgen/demonve

IgorZiegler commented 8 years ago

@jorgenevil i have not account on beta server=)

IgorZiegler commented 8 years ago

@knutole @jorgenevil in this line we have user-a3eef95c-f46a-407c-8109-b62fb36badf2 in userUuid variable, but app.Users[userUuid] doesn't exist.

knutole commented 8 years ago

@IgorZiegler Yes, this is probably due to mismatch between available users and which users have access. (Or perhaps user was deleted...) Can you create a separate issue for this?

IgorZiegler commented 8 years ago

@knutole created here

IgorZiegler commented 8 years ago

@knutole could we check what user is it user-a3eef95c-f46a-407c-8109-b62fb36badf2?

knutole commented 8 years ago

@IgorZiegler Nope, the user is deleted. I think that's the problem here. So solution, anyway, is just to add a check so that there's no error anyway. I will research the underlying issue.

knutole commented 8 years ago

@IgorZiegler Sorry to say, but there keeps popping up more bugs with this. Now the search is not working...

EDIT: NEVERMIND! This bug was introduced by myself in last "fix"... Sorry;)

invite_bug

knutole commented 8 years ago

Fixed some of these bugs, but want to keep this issue open, as I'm not sure all bugs are gone.

Some serious bugtesting needs to be done here.

IgorZiegler commented 8 years ago

@knutole As I see it filter works correctly now.

knutole commented 8 years ago

Closing this, will open new issue if more bugs.