Update workflow variable assignments #431
Merged
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
mitchdawson1982
force-pushed
the
workflow-change
branch
from
480cf58 to
85d1cfb
Compare
tom-webber
approved these changes
Jun 13, 2024
mitchdawson1982
added a commit
that referenced
this pull request
Jun 14, 2024
* add replaces vars with inputs * Remove inputs and pull vars from respective environements
mitchdawson1982
added a commit
that referenced
this pull request
Jun 17, 2024
* add .env.tpl env template file * Add MOJ internal service header (#405) * Add MOJ internal service header The main links are now in a primary nav component. This should go below the phase banner as the banner is supposed to touch the black header. I've also changed the phase from alpha -> beta, and changed the capitalization in the service name. * Remove commented out html * Populate domains drop down with what's been ingested in datahub (#407) * Add missing domain information from charts * Update search tests that hit datahub dev - remove entity which is not currently present - enable the no_duplicates test (we have fixed this) * Load the list of domains from Datahub Previously we hardcoded the list of domains shown in the search filter, and had different lists per environment. This was useful in alpha when we had some junk domains we wanted to filter out, but now we're at a point where every domain in Datahub should be one we want to use. This commit means we now fetch every domain that has something linked to it, and display that in alphabetical order. * Move domain model to models and remove unused model * Refacotr: decouple SearchFacetFetcher from DomainModel * Cache facets fetched from datahub Ideally we would just fetch the facets once per request, but in practice we do this from a few different places. 1. In the view we instantiate a SearchService, which uses the domain model in constructing filters for Datahub. 2. The SearchForm also needs them to know what choices are valid, so we need to pass a callback to the form's ChoiceField. That callback does not share any data with the view. Caching the value is a quick way to avoid making extra requests for the same data. * Hide subdomains if there aren't any defined This is the case at the moment, because the domain model we've pulled in from CaDeT doesn't have subdomains. This might change later though so I don't want to remove the subdomain code completely. * Include missing domains Previously it was only returning domains with tables in. We should include any that show as non-empty in Find MOJ Data. * Cleanup - bring through tags and glossary terms consistently, and remove dead code for data products (#418) * Extend tags and include glossary terms in search results * Remove remaining references to data product This is currently unused, because we no longer include data products in the search. * Set chromedriver path to one installed by setup-chromedriver * Remove metrics ingres config (#425) remove allowed subnets * Show when stuff is an ESDA (#421) * Show when stuff is an ESDA This is only shown on a handful of assets. Also remove metadata fields we have not populated yet (these will always display as blank) * Correct casing * Metrics ingress test (#428) * remove allowed subnets * change ecr_region from input to var * Update workflow variable assignments (#431) * add replaces vars with inputs * Remove inputs and pull vars from respective environements * Fmd 366 add dataset lineage link (#416) * add upstream and downstream lineage to getDatasetDetails graphql query * refactor parse_relations() helper to handle more relations * add upstream and downstream lineage to RelationshipType enum * update parse_relations() input args * update parse_relations() input args in search * add has_lineage and lineage_url to dataset details context * add lineage link to details_table template * remove redundant block in query for data product relationships * return entity name for lineage * have only 1 RelationshipType for lineage * simplfy `parse_relations()` helper function * update DatasetDetails to use single lineage type * align url to rest of table * update tests * add default value for url * design suggestions for lineage label, from Alex and Jess * spell it right * suggestions from Mat * update readme * remove .env.example --------- Co-authored-by: Mat <[email protected]> Co-authored-by: Matt <[email protected]>
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.
No description provided.