-
Notifications
You must be signed in to change notification settings - Fork 0
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
update coordinates response change #397
Draft
soson
wants to merge
89
commits into
main
Choose a base branch
from
FD-602-update-coordinates
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* workflow query definitions * add missing files * better typing of metadata api * schema changes * return tasks as stringified json * workflow node query added
* workflow query definitions * add missing files * better typing of metadata api * schema changes * return tasks as stringified json * workflow node query added * create graphql and io-ts schemas for executed workflow and executed workflow tasks * add filter and pagination query * update calculation of relay specific pagination * add executed workflow detail query * resolve wanted changes to make codebase more readable * remove as type * add tests for conductor helpers (#266) --------- Co-authored-by: Martin Sottnik <[email protected]>
* bump version 1.8.3 (#263) * create pause and resume mutation with bulk ones * add env variable for github action --------- Co-authored-by: Martin Sottnik <[email protected]>
* workflow defintion mutations added * typo fixes; add missing fields * handling bulk result on update workflow * add CONDUCTOR_API_URL to env variables * addWorkflow renamed to createWorkflow
* add mutation to start new workflow * move from executed to workflow file and create start new workflow mutation * add mutation to execute workflow * use our custom logger and use correct name convention * remove trycatches and loggers * add jsdocs and handle JSON parsing * use void instead of custom string and fix errored imports * remove logger
* add workflow filters * move workflow filter to separate function
* add input parameters property to workflow * pr-check fix
* add retry, resume, terminate and remove workflow mutations * remove custom string responses and return isOk * remove redundant trycatch and custom string responses in API calls * rename optional args by our naming conventiosn
* task definitions query added * add missing file; fix type checking * changed task definition props to createdAt, updatedAt * add additional properties
* use executed workflow ids as input in bulk operations * bulk resume fix
* new attributes added to workflow * add output paramters to workflow * add restartable and outputParameters to workflow input * add createdAt, updatedAt to workflow input
* add urql to handle schellar graphql requests * add graphql resolvers for schellar in inventory * add generated types from schellar server * fix hasSchedule logic of workflow * remove urql and use graphql-request * refactor property names * add resolvers for schedule schema properties and refactor query resolvers
* custom connectionFromArray helper function added * remove unused eslint comment
* add subscription for executed workflow * refactor naming and remove uuid * id generated by uuid and workflowId used for mutations
* refactor ID for executed workflow * refactor property naming and url handling of empty value * remove warnings * set default value if priority was not defined * handle optional values from input
* add resource manager to inventory server * push missing files * fix GetPool request
…ventory-server into story-new-conductor-api
* remove conductor,schellar,rm apis * remove env variables from config; remove useless ts-ignore * remove forgotten types in graph id helpers * remove forgotten files
* remove yarn; use npm * add pr check to new branch
* update apollo server to v4 * fix tenantId in context * use default tenantId from config * add @types/cors package; fix type-check error * remove unused functions; refactor
* remove usage of linksAndDevices endpoint * remove hasAndInterfaces endpoint * add idHas to query * remove obsolete hasAndInterfaces/linksAndDevices graphql enpdoints * revert .env.example * idHas removed
plehocky
previously approved these changes
Feb 5, 2024
MarcoMruz
previously approved these changes
Feb 5, 2024
soson
dismissed stale reviews from MarcoMruz and plehocky
February 6, 2024 09:36
The base branch was changed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
HOW TO TEST: test it with topology-discovery v1.3.0-coordinates branch