Skip to content

WIP: Bugfixes

Tristan Le Granché requested to merge strangenoise/stax:bugfixes into 2.0-test

Checklist

Please check if your MR fulfills the following requirements:

  • Unit tests have been added, if needed
  • Documentation have been updated, if needed
  • Build was run locally and any changes were pushed
  • Lint has passed locally without error
  • Package.py have been updated, if needed
  • CHANGELOG.md have been updated, if needed
  • Version numer has been tagged, if needed

Merge request type

Please check the type of change your MR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

Breaking change?

  • Yes
  • No

Summary

  • Authentication and scene.tracks errors (#104 (closed))
  • Startup errors when modules are not installed (#103 (closed))
    • Doesn't fix the main problem, but fix some errors and cleaner
  • Update workspaces names across all files

More to come

Description of change

  • Authentication and scene.tracks errors

    • Authentication raises an error only if we've got a linker authentication fail

    • Otherwise raises a warning saying that no linker were found and stax will work without any linker

    • Check if scene.tracks as a length before trying to use it.

  • Startup errors when modules are not installed

    • Give the user the information that Blender needs to be relaunched
    • Correct an error using pip with a Path instead of a string
    • Remove the install_lib() from the try of openreviewio, as it needs to be launched only in the except part.
  • Update workspaces names across all files

    • Replace ["Player Mode"] with ["Main"]
    • Replace ["User Configuration"] with ["Stax configuration"]
    • Replace ["Export Mode"] with ["Timeline export"]

How to test it

Review

  • Assigned To: @Tilix4
  • Expected review time: 2 days
  • FYI: @awesome_dev_too
Edited by Tristan Le Granché

Merge request reports