Skip to content

Merge app builder workflow branch to dev#472

Merged
jwildfire merged 5 commits intodevfrom
dev-workflow
Feb 3, 2021
Merged

Merge app builder workflow branch to dev#472
jwildfire merged 5 commits intodevfrom
dev-workflow

Conversation

@xni7
Copy link
Contributor

@xni7 xni7 commented Jan 8, 2021

@jwildfire @sibyllwangxxxx PR for #427. Also see 08JAN2021 V2 tech team meeting discussion #471

Copy link
Contributor

@jwildfire jwildfire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good @xni7! I was able to run the plug-in with no problem and the viewer for the chart metadata in the main app is much improved and works well.

As discussed, I'm going to go ahead and merge this to dev with the understanding that some more work is needed on this set of functionality before v2.0 release. Maybe the next step could be to go ahead and update the chart creation vignette to describe how these new tools will help users with chart creation and provide some sample code and instructions? Then we can file some issues to start fine tuning things.

@jwildfire jwildfire merged commit 300bdd6 into dev Feb 3, 2021
@jwildfire jwildfire deleted the dev-workflow branch February 3, 2021 17:17
@xni7
Copy link
Contributor Author

xni7 commented Feb 6, 2021

Agreed @jwildfire . Just saw your comment. , let's update the vignette to describe the high level workflow based on this prototype. With the design thinking, it may be helpful get feedback from Mengchun, Jim, et al on the prototype next, as part of alpha testing.

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.

2 participants