Skip to content
This repository has been archived by the owner on May 11, 2021. It is now read-only.

Add parameterization and online library import export #138

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sudo-panda
Copy link
Contributor

@sudo-panda sudo-panda commented Sep 1, 2019

This pull request will : -

  • Add parameterization to VisualStates tool which includes:

    • A tab in Local Namespace dialog to Add/Edit/Remove parameters.
    • Saving of parameters during saving of VisualStates File.
    • Loading of parameters when opening VisualStates File.
    • Importing parameters when importing VisualStates File.
    • Replacing parameters with their values when generating Python/C++ Code.
  • Add import export dialog for the online library which does the following:

    • When user clicks on File > Library Export it asks user for his GitHub username and password
    • Once the username and password have been verified it asks the user to input the name of the behaviour and a description of it.
    • It then connects to the library repository and creates a fork on the users accounts or uses the already existing one.
    • It syncs the master branch of the fork to the library via a pull request which is automatically merged.
    • Then it checks for an existing branch of the same name as to be created, if a branch exists it writes the files and updates the catalogue on it or else it creates a branch of that name and writes to the one that has been created.
    • Lastly it checks if a pull request is present from the branch to the library and if there isn't one it creates one itself.
    • In the case of Import, it lists the available behaviours and the user and cancan select between them
    • Once the user has selected the behaviour it downloads the file and imports it into the tool.

Explanation Videos:

Links:

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant