qgis-plugin icon indicating copy to clipboard operation
qgis-plugin copied to clipboard
verified publisher icon MerginMaps

Workspaces

Open uclaros opened this issue 3 years ago • 3 comments

Requires https://github.com/MerginMaps/mergin-py-client/pull/150

Changes in classes class MerginGroupItem(QgsDataCollectionItem): and class MerginRootItem(QgsDataCollectionItem): are easier to review side-by-side before and after, the diff is messy.

If server is old and does not support workspaces, the groups in browser are reduced to two: My projects and Shared with me: app dev

If server is CE, there are no groups and the global namespace appears in brackets: ce dev

If server is EE, the active workspace is displayed in brackets and an option to switch workspace is offered: Peek 2022-10-26 13-07

Switching between old/ce/ee servers should be transparent to the user: Peek 2022-10-26 13-09

When there are no workspaces the user gets a warning with a link to mergin: image

~Realized while taking screenshots:~ Fixed

  • [x] ~Apparently, when switching back to EE server, the last used workspace is not preserved, will investigate.~
  • [x] ~Mergin Maps [None] looks ugly, should probably remove brackets when there is no workspace~
  • [x] ~Create new project should not be available when there is no workspace~

uclaros avatar Oct 26 '22 07:10 uclaros

👏 can you please add some screenshots/gifs?

PeterPetrik avatar Oct 26 '22 09:10 PeterPetrik

clap can you please add some screenshots/gifs?

Done (and found a couple of bugs while doing so)

uclaros avatar Oct 26 '22 10:10 uclaros

Task linked: CU-24g3jrv Adapt plugin to workspaces

ghost avatar Oct 28 '22 08:10 ghost

image

uclaros avatar Nov 02 '22 14:11 uclaros

Non writable workspaces show an error message

Peek 2022-11-03 13-03

Peek 2022-11-03 13-27

uclaros avatar Nov 04 '22 15:11 uclaros

@varmar05 you have unresolved review comments here. Can you check them, please?

tomasMizera avatar Nov 21 '22 07:11 tomasMizera