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

Operations Graph #15

Merged
merged 9 commits into from
Nov 1, 2023
Merged

Operations Graph #15

merged 9 commits into from
Nov 1, 2023

Conversation

JamieScottC
Copy link
Collaborator

So this branch is still a WIP, but I thought it might be a good idea to merge it into master before the meeting on Wednesday so we can get some feedback on the Graph's UI even though it's not functional yet. What do you think?

It also has some general bug fixes that came up while implementing the graph that are quite important, and it gives both of us the opportunity to iterate on the graph if we want. I know it's a big one so here's a list of changes that can be tested:

  • Implemented "graph outline" in operations page
  • Fixed weird edit ability behavior where executors were disappearing after closing and reopening ability modal
  • Added "operation status" widget on home page
  • Fixed code editor bug where input would not be registered after switching between operations page and ability modal
  • Added "v-network-graph" and "d3-force" packages for graph view

Also make sure to npm install again because of the new packages

@JamieScottC JamieScottC requested a review from morinjmit October 30, 2023 19:39
@morinjmit
Copy link
Contributor

Looks awesome!

Definitely like the idea of merging it into so people can play with it and try it out and give feedback

Are we able to merge it into a different branch that they can switch to and still. see it? Or does it have to be master?

If master maybe we can have a popup or disclaimer saying this is still a work in progress and not complete

@JamieScottC JamieScottC merged commit 1b93f27 into master Nov 1, 2023
1 check passed
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