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

Manual test - first version of process editor #11504

Closed
13 tasks done
Tracked by #11501 ...
nkylstad opened this issue Nov 5, 2023 · 2 comments
Closed
13 tasks done
Tracked by #11501 ...

Manual test - first version of process editor #11504

nkylstad opened this issue Nov 5, 2023 · 2 comments
Labels
area/process Area: Related to app process (e.g. signing, receipt, fill inn, payment, etc).

Comments

@nkylstad
Copy link
Member

nkylstad commented Nov 5, 2023

Description

Perform a manual test of the process editor.

Scope

This task should only test the process editor itself. This includes adding/removing/configuring process within the BPMN editor.

Acceptance criteria

Apps with v7 or earlier of app-backend:

  • Should be shown an info message that they cannot edit the process
  • Should see their process
  • Should be able to zoom/move process
  • Should be able to select a task

Apps with v8 or later of app-bakend:

  • Should see their process
  • Should see the process palette
  • Should be able to add a task from the process palette
  • Should be able to rename a task in the editor
  • Should be able to delete a task via the context menu
  • Should be able to move tasks around in the diagram
  • Should be able to move connectors around in the diagram
  • Should be able to save changes
  • Saved changes should be persisted on re-load
@nkylstad nkylstad added the area/process Area: Related to app process (e.g. signing, receipt, fill inn, payment, etc). label Dec 18, 2023
@mlqn mlqn self-assigned this Jan 3, 2024
@mlqn
Copy link
Contributor

mlqn commented Jan 4, 2024

I have tested it in both dev and prod, and everything works as expected 🥳
I tested version 8 by both upgrading the Altinn.App.Api version in the App.csproj file, and by using the shouldOverrideAppLibCheck featureFlag.
I just noticed a small display error that might be a bit confusing for the user: Dragging a task increases the counter for unsaved changes

@mlqn mlqn removed their assignment Jan 4, 2024
@nkylstad
Copy link
Member Author

nkylstad commented Jan 8, 2024

I just noticed a small display error that might be a bit confusing for the user: Dragging a task increases the counter for unsaved changes

Nice! Technically, changing the diagram does also change the bpmn-file, but I agree that it might be strange for the user. Not a blocker for the process editor though, so we can consider this issue as input going forward!

@nkylstad nkylstad closed this as completed Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/process Area: Related to app process (e.g. signing, receipt, fill inn, payment, etc).
Projects
Archived in project
Development

No branches or pull requests

2 participants