2064 Commits

Author SHA1 Message Date
Kevin Burnett
a5e5ad56ad Merge pull request #157 from sartography/orjson
poetry remove orjson
2023-02-27 06:47:19 -08:00
Jon Herron
a766e7d9a6 poetry remove orjson 2023-02-27 09:35:51 -05:00
burnettk
e9b3ababc1 wip for get_last_user_completing_task script task 2023-02-25 23:31:23 -05:00
burnettk
f7dc076f75 postgres really will just order however it wants if you do not specify an order_by clause 2023-02-25 22:35:46 -05:00
burnettk
647d5ee92a add ppg.ba4.sme and ba5 2023-02-24 23:42:19 -05:00
burnettk
5eb9da00f1 fix postgres db name and comment out debug job 2023-02-24 16:57:53 -05:00
jasquat
ff1df1145b lint w/ burnettk 2023-02-24 16:29:02 -05:00
jasquat
b71419616d removed some unused code from task and fixed the logs table a bit w/ burnettk 2023-02-24 16:18:49 -05:00
Dan
ff069cb862 run_pyl 2023-02-24 15:02:03 -05:00
jasquat
425cf3f8c1 skip failing test if postgres and added comment about cause w/ burnettk 2023-02-24 14:57:17 -05:00
Dan
0dc2bc3316 # SpiffWorkflow:
1) Type Safe checking on correlation properties (no more str())
2) A running workflows Correlations are once again at the key level.

# Backend
1) Both send and receive messages can have correlation_keys - and we compare these to each other to quickly assure a match (if they both exist - otherwise we fall back to comparing the properties on the receive to the sending messages payload)
2) Cleaned up the migrations to just one file
2023-02-24 14:53:22 -05:00
jasquat
7e1f7942c3 added remaining dependabot items and commented them out and an auto merger w/ burnettk 2023-02-24 14:40:09 -05:00
dependabot[bot]
daf920666b Bump actions/upload-artifact from 2 to 3
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 2 to 3.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/v2...v3)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
2023-02-24 19:31:39 +00:00
burnettk
9a466ab65e commit dependabot from backend 2023-02-24 14:31:10 -05:00
jasquat
c66a078879 updated actions checkout in images as well w/ burnettk 2023-02-24 14:27:59 -05:00
jasquat
349bba87e9 updated actions checkout to remove deprecations warnings w/ burnettk 2023-02-24 14:14:17 -05:00
jasquat
4c9fa21fe4 set-output is deprecated w/ burnettk 2023-02-24 14:00:21 -05:00
jasquat
4bbb714792 added some examples and print the tags after creating the docker images w/ burnettk 2023-02-24 13:44:21 -05:00
jasquat
6e03c264ca added github action to push images for main branch on all builds w/ burnettk 2023-02-24 11:42:34 -05:00
jasquat
d5712a915c look users up by service and username instead of service_id since usernames have to be unique anyway w/ burnettk 2023-02-24 10:41:57 -05:00
jasquat
144c65e4b7 Merge pull request #147 from sartography/feature/task_data_api_refactor2
avoid using task-data endpoint for task data and only use it to get t…
2023-02-24 10:07:34 -05:00
jasquat
7a1e712c14 avoid using task-data endpoint for task data and only use it to get tasks based on spiff step instead 2023-02-24 09:21:21 -05:00
jasquat
4651eaee94 put back the task data code when getting tasks 2023-02-23 18:32:59 -05:00
jasquat
0b44f76581 added some docker tag notes to github action file w/ burnettk 2023-02-23 17:16:35 -05:00
jasquat
f1e82c564f Merge pull request #146 from sartography/feature/task_data_api_refactor
Feature/task data api refactor
2023-02-23 17:10:47 -05:00
jasquat
1f95fe7037 Merge remote-tracking branch 'origin/main' into feature/task_data_api_refactor 2023-02-23 17:10:23 -05:00
jasquat
d53b79b18a removed commented out code w/ burnettk 2023-02-23 17:09:33 -05:00
jasquat
a4c1eb96b5 removed task-data endpoints since we no longer need them w/ burnettk 2023-02-23 17:07:53 -05:00
burnettk
441bd62652 turn off profiling again 2023-02-23 16:54:38 -05:00
burnettk
5c43967ee9 add two users and update one 2023-02-23 16:39:30 -05:00
Dan
5a3b4b8792 BPMN Parser was returning all retrieval expressions, rather than the ones specific to a correlation property, as was intended.
Adding a correlation cache - so we have a reference of all the messages and properties (though still lacking a description of keys)
Adding yet another migration, maybe should squash em.
2023-02-23 16:36:32 -05:00
jasquat
a9694c7678 added api to get task data and do not return from task data list anymore w/ burnettk 2023-02-23 15:59:19 -05:00
Dan
0d468a825a run_pyl (part 2) 2023-02-23 15:09:22 -05:00
jasquat
15b00bcceb Merge branch 'main' of github.com:sartography/spiff-arena 2023-02-23 15:06:27 -05:00
jasquat
68a5fe3f16 added comment about refactoring getting task data w/ burnettk jbirddog 2023-02-23 15:06:23 -05:00
burnettk
ab631aab99 lint 2023-02-23 15:01:45 -05:00
burnettk
973be69df5 add test users 2023-02-23 14:58:14 -05:00
Dan
8466d823c2 Adding Migration. 2023-02-23 14:42:00 -05:00
jasquat
49fb41eff9 remove task size check since it can take a long time to run and we do not do anything with it w/ burnettk jbirddog 2023-02-23 14:41:45 -05:00
Dan
7c12dffe41 Merge remote-tracking branch 'origin/main' into feature/message_fixes 2023-02-23 14:26:04 -05:00
Dan
0030a46938 run_pyl 2023-02-23 14:17:22 -05:00
Dan
0f3ef00d72 BPMN.io -- Just show the message names not the ids - to assure we are only exposing the names.
SpiffWorkflow -
    - start_messages function should return message names, not ids.
    - don't catch external thrown messages within the same workflow process
    - add an expected value to the Correlation Property Model so we can use this well defined class as an external communication tool (rather than building an arbitrary dictionary)
    - Added a "get_awaiting_correlations" to an event, so we can get a list of the correlation properties related to the workflows currently defined correlation values.
    - workflows.waiting_events() function now returns the above awaiting correlations as the value on returned message events
 Backend
    - Dropping MessageModel and MessageCorrelationProperties - at least for now.  We don't need them to send / receive messages though we may eventually want to track the messages and correlations defined across the system - these things (which are ever changing) should not be directly connected to the Messages which may be in flux - and the cross relationships between the tables could cause unexpected and unceissary errors.  Commented out the caching logic so we can turn this back on later.
    - Slight improvement to API Errors
    - MessageInstances are no longer in a many-to-many relationship with Correlations - Each message instance has a unique set of message correlations specific to the instance.
    - Message Instances have users, and can be linked through a "counterpart_id" so you can see what send is connected to what recieve.
    - Message Correlations are connected to  recieving message instances.  It is not to a process instance, and not to a message model.  They now include the expected value and retrieval expression required to validate an incoming message.
    - A process instance is not connected to message correlations.
    - Message Instances are not always tied to a process instance (for example, a Send Message from an API)
    - API calls to create a message use the same logic as all other message catching code.
    - Make use of the new waiting_events() method to check for any new recieve messages in the workflow (much easier than
    churning through all of the tasks)
    - One giant mother of a migration.
2023-02-23 13:53:03 -05:00
jasquat
216bd799bf do not call serialize if we can use the cached bpmn_json instead w/ burnettk 2023-02-23 13:48:27 -05:00
jasquat
7dd72c72e1 Merge branch 'main' of github.com:sartography/spiff-arena 2023-02-23 11:19:26 -05:00
jasquat
2bf31fc135 turn on sentry detailed tracing for task-data w/ burnettk 2023-02-23 11:19:22 -05:00
burnettk
55f3b765ee update spiff 2023-02-23 11:02:40 -05:00
burnettk
3e374f93cb Squashed 'bpmn-js-spiffworkflow/' changes from 9252af81..f1f008e3
f1f008e3 fixing a bug in the bpmn editor that was attempting to create the wrong element.

git-subtree-dir: bpmn-js-spiffworkflow
git-subtree-split: f1f008e3e39be43b016718fca6a38b248ab4ecf7
2023-02-23 10:50:00 -05:00
burnettk
480b61ad59 Merge commit '3e374f93cb906e361c80f407dd15850e269473ae' 2023-02-23 10:50:00 -05:00
burnettk
ce1e54dced Squashed 'spiffworkflow-frontend/' changes from f099c181..c22d22ba
c22d22ba logs list page should respect the for-me variant
dcc3e3b1 show help text for textareas as well w/ burnettk
35b8bf35 default to the for-me path on process instance show page links
da646e54 added support to validate custom errors in nested properties in json schema forms
17412e9d make it so the message at the top of form is never strictly wrong
7dea59b7 adding some padding to form text inputs w/ burnettk
3f92f492 added ability to display the environment in the frontend header bar w/ burnettk
85246120 put the env vars in the env section of the github action configs w/ burnettk
58ff9075 added test for quickstart guide w/ burnettk
03e725b0 corrected cypress env var
db755731 added some support for using the backend openid server for cypress tests w/ burnettk
e76b6429 anything in the Tasks waiting for me table can now be completed by the current user
56403b4c users can always complete tasks on process instance show page and on task group table on home page w/ burnettk
e2ab6eff avoid endless redirects on error on authentication list page w/ burnettk
66608837 bug fix
4781d5fc link to the spiff step from a task on the frontend and use the correct db in ci
69eb426c pyl w/ burnettk
3cf036b0 call activities are also working w/ burnettk
40b101a7 add assertions so this fails fast if anything changes in the future
74d2d3fa Update README.md
24c3c431 fix formatting
ed977079 document runtime config
e62427b9 do not fail if SPIFFWORKFLOW_FRONTEND_RUNTIME_CONFIG is missing
551bdf17 allow setting configs for the frontend through env vars w/ burnettk
bc91c62c Merge remote-tracking branch 'origin/main' into frontend/use-api-subpath
eeab0d0f Use the same markdown library for displaying as for editing - could enable a security plugin, but doing so would prevent BPMN developers from using the Sub and Sup markdown supported by GitHub.
28d6f75c When searching for human tasks to determine if the current user can complete it, filter on the "completed" flag.
34b36fe2 put setDisabled back in the awkward place since i was seeing the subsequent form stay disabled
4794a8cd Merge branch 'main' of github.com:sartography/spiff-arena into main
78b9ef73 Fix that dreadful unknown "KeyError" exception that was cropping up. Adding a bit of detail to the spiffworkflow exceptions when a duplicate process model is found. Disable the submit button on tasks after you click submit (avoid the double click and give users a better experience)
d854a122 Merge pull request #125 from sartography/feature/dynamically-hide-fields-w-task-data
eda73854 make form schema and form ui schema both dicts, add support for hiding fields based on task data
bda549f3 Prevent double click on submit of forms.
3f02fa80 try to improve exception handling by avoiding raising ApiError from services
06835267 remove duplicate label on radio buttons
5d1e3517 less annoying file name for autocomplete
c569acde frontend: use /api subpath instead of subdomain
d3dda548 add deps for serve
e6173efe Revert "revert Dockerfile until we get it working"
74103b97 revert Dockerfile until we get it working
8e9badd3 get bin as well for script
df7a97e6 Merge pull request #116 from sartography/frontend/improve-dockerfile
7a17bc6f remove unneeded divs
2cf58c24 IBM says you can't have more columns than your parents, even if you try to start another grid, with kburnett
abc85ee3 make task show wide, and make repeating form icons match site styles. w/ dfunk
6400827e even textareas need to have blank labels since labels are in FieldTemplate
462852d6 replace fieldTemplate with unthemed core version and remove labels since that is handled in there
a50dd004 wrap field template so we can style with margin bottom
e6a1ca97 new mechanism to handle help more in line with how carbon works
cb6ce6b4 frontend: avoid redundant steps in Dockerfile
b6b7ceeb we were expecting an object when doing this check, so codify it
cfcfe292 use the 403 response to tell if a user has access to task data on the task show page w/ burnettk
355406e2 do not reset error state from the adminroutes component since this causes the error to be removed right away and then cannot be displayed w/ burnettk
3e7962cf if we get a result back it will be a task with a model identifier
0f043f72 use the ProcessInstanceTask interface where we can and move some stuff around better for useEffect
4ee4e5e0 run_pyl had various recommendations that I find a bit of a pain in the butt, but that I did anyway.
84e7d3fa Merge remote-tracking branch 'origin/main' into feature/jinja_errors
acbc0a07 Added useMemo to error context No longer clear errors in the task bar, as that will constantly remove them as soon as they are shown.
19c3ab0e TaskShow had a useEffect that depended on params, that dependency caused an infinite request cycle when an error occured. The same issue was happening on the ProcessInstanceListTable, and there it was being managed by a "SafelySetErrorMessage" function in one case, but would not be addressed in all possible cases.
c37224ec added the process model identifier for the diagram if it is not the top level w/ burnettk
befefb6d Merge pull request #107 from sartography/feature/metadata_on_instance_show
65e42110 use a modal for metadata instead w/ burnettk
4a2f2a00 put process instance show page to match main w/ burnettk
0f1ab490 do not allow deleting primary bpmn file and do not allow instantiating models without a primary bpmn file w/ burnettk
3718f1a3 show metadata on instance show page but for some reason it reorders elements w/ burnettk
400a1469 add process model file name validation for new files w/ burnettk
132c5814 added locking system for process instances so hopefully background jobs will not take instances currently being run by the user w/ burnettk
3fa86949 make sure that all new form field elements are not dropdowns
f69d3416 expanded functionality of the form builder
2453938f use the correct place for keycloak w/ burnettk
3d85a5f6 Merges
50d0a8e3 Lots of adjustments from running pyl Main change is in the ErrorDisplay.tsx to assure all error information is provided. and index.css to make it "pretty"

git-subtree-dir: spiffworkflow-frontend
git-subtree-split: c22d22ba5d8a96dca66a405d9c230284c81b4740
2023-02-23 10:49:57 -05:00
burnettk
1877aac9be Merge commit 'ce1e54dced72efbc5ffb7da3b6699413bceb5e39' 2023-02-23 10:49:57 -05:00