Remove har to yaml from controller#272
Merged
Merged
Conversation
burnsidepj
approved these changes
Dec 6, 2024
tkmcmaster
added a commit
that referenced
this pull request
Dec 6, 2024
Merged
tkmcmaster
added a commit
that referenced
this pull request
Dec 6, 2024
* Fix multiple test results bug (#269) * Fixed the bug with multiple results files and switching between - The null pointer throws on redraw. All the calculations and freeing of memory must be done within a setState block while the data is static * Updated eslint to include storybook * Updated eslint to include storybook * Fixed eslint files * Michael/perf 1699 (#270) * Updated Request tool to get it online * Removed unused component * Updates to remove lint errors * PewPew improvements (#271) * PewPew improvements * Updated scripts for running locally * Remove har to yaml from controller (#272) * Removed the yamlwriter and all its components from the controller * Changed the yamlwriter link in the layout to go to the guide yaml writer * Removed Non Prod message for login * Removed unneeded vars --------- Co-authored-by: blonde-mike <macrowther7+git@gmail.com>
tkmcmaster
added a commit
that referenced
this pull request
Jan 6, 2025
* Removed the yamlwriter and all its components from the controller * Changed the yamlwriter link in the layout to go to the guide yaml writer * Removed Non Prod message for login * Removed unneeded vars
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Summary: Removed the Yaml Writer from the controller and changed the
Layoutlink to point to the yamlwriter in the guide. No more duplication!