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

Fix partial Workspaces export #99

Closed
wants to merge 2 commits into from

Conversation

druckgott
Copy link

@druckgott druckgott commented Apr 9, 2024

Workspace Export now works. It's only one line to fix it.

Also, an example export:
workspaces.json

Currently there is one problem on the beginning you have to insert: {graphConfig : [ and on the end ]}
And then ZIP the file.

You can also manually do it until the new version is coming out:

got to main.js

search for:

            workspaceGraphConfigs = {graphConfig : [
                                    null,null,null,null,null,null,null,null,null,null
                                    ]};
                                    
replace with

workspaceGraphConfigs = graphConfig;

Fix Workspace Export
@Tonynux
Copy link

Tonynux commented Apr 12, 2024

Hi. This is my update in main.js :

// New workspaces feature; local storage of user configurations
prefs.get('workspaceGraphConfigs', function(item) {
    if(item) {
        workspaceGraphConfigs = item;
        } else {
        //workspaceGraphConfigs = {graphConfig : [
        //                        null,null,null,null,null,null,null,null,null,null
        //                        ]};
        //Tony
  workspaceGraphConfigs = graphConfig;
        }
});

// Get the offsetCache buffer

Now I created my Graphs:

Graph
Legend

Then trying to export. And this is my JSON file:

json

Just one field exported.

@druckgott
Copy link
Author

Please try to press Ctrl z 2 Times and test it again please.

@Tonynux
Copy link

Tonynux commented Apr 12, 2024

Pressed Ctrl z 2 times (before exporting workspace), but... same result: one field exported.

@druckgott
Copy link
Author

Mhh. Very Strange i think there is a bigger Bug inside.

@mmosca
Copy link
Collaborator

mmosca commented Aug 14, 2024

Can you update to the latest master, so it runs the ci build?

@druckgott
Copy link
Author

Done

@druckgott druckgott closed this by deleting the head repository Oct 5, 2024
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.

3 participants