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

Neat 68 update information architect text #324

Merged
merged 15 commits into from
Mar 18, 2024

Conversation

nikokaoja
Copy link
Collaborator

@nikokaoja nikokaoja commented Mar 18, 2024

  • updated mock graph step
  • Finilize Info Architect text
  • Update text up to DMS Architect
  • Created videos and uploaded them to youtube
  • Added videos to text up to DMS Architect

UI improvements:

  • enabled right click
  • workflow setting are saved prior workflow start
  • filtering added on rules to show only XLSX files
  • removed bunch of legacy console.logs

@nikokaoja nikokaoja requested a review from doctrino as a code owner March 18, 2024 11:49
@@ -565,7 +565,7 @@ return (
</Panel>
</ReactFlow>

<Button variant="outlined" onClick={startWorkflow} sx={{ marginTop: 2, marginRight: 1 }}>Start workflow</Button>
<Button variant="outlined" onClick={ () => {saveWorkflow(); startWorkflow()} } sx={{ marginTop: 2, marginRight: 1 }}>Start workflow</Button>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sweet!

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it was easier than expected, but took a while to find out where and how to change this

Copy link
Collaborator

@doctrino doctrino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Way to go! Love the extra effort with youtube videos :)

And nice with so many small annoyances gone.

to specify where the statement comes from, or what standard that matches the statement. The second column is called
`MatchType`, and tells whether the source is partially or fully matching the statement.
The process of configuration and execution of these steps is shown in the video below:
https://youtu.be/_iTQatLBpns
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this also be put into an iframe?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it should not be there at all :D

Co-authored-by: Anders Albert <60234212+doctrino@users.noreply.github.com>
Co-authored-by: Anders Albert <60234212+doctrino@users.noreply.github.com>
@nikokaoja nikokaoja merged commit 5f6f4e2 into main Mar 18, 2024
6 checks passed
@nikokaoja nikokaoja deleted the NEAT-68-Update-Information-Architect-Text branch March 18, 2024 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants