123 Commits (49b773233b80d42941dbfdc0a645e75972c2eec0)

Author	SHA1	Message	Date
John Cowen	25f989753b	ui: Adds @NullValue attr decorator (#9587) ... There are many places in the API where we receive a property set to `null` which can then lead to defensive code deeper in the app in order to guard for this type of thing when usually we are expecting an array or for the property to be undefined using omitempty on the backend. Previously we had two places where we would deal with this in the serializer using our 'remove-null' util (KV and Intentions). This new decorator lets you declaritively define this type of data using a decorator @NullValue([]) (which would replce a null value with []. @NullValue in turn uses a more generic @replace helper, which we currently don't need but would let you replace any value with another, not just a null value. An additional benefit here is that the guard/replacement is executed lazily when we get the property instead of serializing all the values when they come in via the API. On super large datasets, where we only visualize part of the dataset (say in our scroll panes), this feels like a good improvement on the previous approach.	4 years ago
John Cowen	92f0eb3bdc	ui: Re-organize our %h* placeholders (#9584) ... We've always had this idea of being able to markup up information semantically without thinking about what it should look like, then applying our %h* placeholder styles to control what the information should look like. Back when we originally made our set of %h* placeholders, we tried to follow Structure as much as possible, which defined the largest header (which we thought would have been the h1 style) as a super large 3.5rem. Therefore we made our set of %h* placeholders the same as Structure beginning at a huge 3.5 size. We then re-overwrote those sizes only in Consul specific CSS files thinking that this was due to us existing before Structure did. Lately we saw an extra clue in Structure - the extra large 3.5 header was called 'h0'. This commit moves all our headers to use a zero based scale, and additionally uses our 3 digit scale as opposed to 1 digit (h1 vs h100), similar to our color scales (note we don't use a hypen, which we can alter later if need be), which means we can insert additional h150 etc if need be. Additional we stop styling our headers globally (h1 { @extend %h100; } ). This means there is no reason not to use headers for marking up content depending on what it is rather than what it should look like, and as a consequence means we can be more purposeful in ordering h* tags. Lastly, we use the new scale over the entire codebase and update a couple of places where we were using using header tags due to what the styleing for them looked like rather than what the meaning/order was.	4 years ago
John Cowen	82a62cd2ed	ui: Sidebar navigation / redesign (#9553) ... * CSS for moving from a horizontal main menu to a side/vertical one * Add <App /> Component and rearrange <HashcorpConsul /> to use it 1. HashicorpConsul now uses <App /> 2. <App /> is now translated and adds 'skip to main content' functionality 3. Adds ember-in-viewport addon in order to visibly hide main navigation items in order to take them out of focus/tabbing 4. Slight amends to the dom service while I was there	4 years ago
Jeff Escalante	88d5e00f99	Update UI staging build for vercel hosting (#9628) ... * update ui staging for vercel * move vercel.json * add a couple more settings * fix comments	4 years ago
John Cowen	bb95738321	ui: Search/filtering 'Filtered by:' search status (#9442) ... Adds a 'status' for the filtering/searching in the UI, without this its not super clear that you are filtering a recordset due to the menu selections being hidden once closed. You can also use the pills in this status view to delete individual filters.	4 years ago
John Cowen	02772e46a2	Fix -ui-content-path without regex (#9569) ... * Add templating to inject JSON into an application/json script tag Plus an external script in order to pick it out and inject the values we need injecting into ember's environment meta tag. The UI still uses env style naming (CONSUL_*) but we uses the new style JSON/golang props behind the scenes. Co-authored-by: Paul Banks <banks@banksco.de>	4 years ago
Kenia	ffb6680e76	ui: Removing formatting to display LockDelay in nanoseconds (#9594) ... * Removing formatting to display LockDelay in nanoseconds * Update test * Add changelog	4 years ago
John Cowen	30014ff8fe	ui: Convert Service.GatewayConfig to a model fragment (#9586) ... * ui: Convert Service.GatewayConfig to a model fragment We added the ember-intl addon, which has its own format-number helper. We replaced our own similarly named helper with this one, but the ember-intl one is far stricter and errors if the arguments passed are undefined. Our previously one would cope with this. We'd rather continue to use the stricter ember-intl helper, so here we convert the GatewayConfig property to a model fragment so that we can give the GatewayConfig.AssociatedServices property a default zero value.	4 years ago
John Cowen	be694366a7	ui: Topology intention saving improvements (#9513) ... * ui: Keep track of existing intentions and use those to save changes Previously we risked overwriting existing data in an intention if we tried to save an intention without having loaded it first, for example Description and Metadata would have been overwritten. This change loads in all the intentions for an origin service so we can pick off the one we need to save and change to ensure that we don't overwrite any existing data.	4 years ago
John Cowen	41a4a9f4fb	ui: Improves the 'you must provide ... to `generate`' error from href-to (#9514) ... This originally comes form the ember-href-to helper and is one of those errors that when I see it I think ... hmmm This gives a little bit more of a clue as to what is wrong by logging the route name you asked for plus the params you passed to it so you: 1. Have more help finding the href-to that is problematic in the template/component 2. Can see all the parameters you passed (including a potential null parameter for the thing you are missing)	4 years ago
John Cowen	1e30503ece	ui: Show a feedback icon instead of a learn icon in the help menu (#9552)	4 years ago
Kenia	b86eea4fba	ui: Rename a model attribute to not be overwritten by ember-data (#9524) ... * Rename a model attr to not be overwritten by ember-data * Make sure we can click on the instances * Make sure we can click back to the preevious page, not root * Add a forwards/back/forwards navigation test for service instances * Rename a model attr to not be overwritten by ember-data Co-authored-by: John Cowen <jcowen@hashicorp.com>	4 years ago
John Cowen	1304dc8820	ui: Adds localStorage to <html class=""> mapping for user controlled theming (#9496)	4 years ago
John Cowen	a42e844ccc	ui: Add EmptyState for exposed paths, plus additional details (#9445) ... * ui: Add EmptyState for exposed paths, plus additional details 1. Add the port to the combined address 2. Use the proxy address for the ip address used * Convert to glimmer component * Add spaces to ListenerPort and LocalPathPort Tooltips	4 years ago
John Cowen	17438020f1	ui: Accessibility scan improvements (#9485) ... * ui: Remove all vestiges of role=tabpanel * Switch out tablist role for a label, default to Secondary * Move healthcheckout-output headers to h2, ideally these would be outside the component * Add aria-label for empty button * Fix up non-unique ids in topology component * Temporarily fixup h2 in KV > LockSession * Fixup dl with no dt * h3 > h2 * Fix up page objects that were reliant on ids	4 years ago
John Cowen	d0ebb2b774	ui: Fix up erroneously placed conditional (#9447) ... This meant that when where were no upstreams, no message would show telling you of that fact. This commit means an EmptyState will now show instead.	4 years ago
John Cowen	8c9d5ecc2b	ui: [BUGFIX] Role/Policy selector default sort (#9434) ... * Sort the options when they aren't being searched	4 years ago
John Cowen	4480302883	ui: [BUGFIX] Request intention listing with ns parameter (#9432) ... This PR adds the ns=* query parameter when namespaces are enabled to keep backwards compatibility with how the UI used to work (Intentions page always lists all intention across all namespace you have access to) I found a tiny dev bug for printing out the current URL during acceptance testing and fixed that up while I was there.	4 years ago
John Cowen	8c0473a622	ui: [BUGFIX] Ensure namespace is used for node API requests (#9410) ... Nodes themselves are not namespaced, so we'd originally assumed we did not need to pass through the ns query parameter when listing or viewing nodes. As it turns out the API endpoints we use to list and view nodes (and related things) return things that are namespaced, therefore any API requests for nodes do require a the ns query parameter to be passed through to the request. This PR adds the necessary ns query param to all things Node, apart from the querying for the leader which only returns node related information. Additionally here we decided to show 0 Services text in the node listing if there are nodes with no service instances within the namespace you are viewing, as this is clearer than showing nothing at all. We also cleaned up/standardized the text we use to in the empty state for service instances.	4 years ago
John Cowen	99f102705d	ui: Fuzzy and Regex searching (#9424) ... Moves search things around to match an interface that can be switched in and out of fuzzy searching using fuse.js. We add both fuzzy searching and regex based searching to the codebase here, but it is not yet compiled in.	4 years ago
John Cowen	25d6a1277e	ui: Use InformedAction for KV table row delete confirmations (#9425)	4 years ago
Kenia	c7d9177775	ui: Add Local flag to local dc in dropdown (#9419) ... * Add Local flag to local dc in dropdown * Remove brand color from the check * Move styling to main-nav-horizonal styling sheets	4 years ago
John Cowen	2e3a66efb7	ui: Misc changes for Catalog area (#9414) ... * Use DataLoader errors for Service Detail and Service Instance * uiCfg > config use the repo-like async interface where possible * Clean up node show * Make sure you can put `=` in dev cookie values * Never default to default * Tweak chain variable * Remove env service * Pass chain through to the template for the tempalte to clean it up * Delete controller tests * Remove cleanup in Nodes show as this is still being used in another tab * Use dc.Local	4 years ago
John Cowen	635cf4dc93	ui: Change references to L7 to say Layer 7 (#9412)	4 years ago
John Cowen	66cc91c69a	ui: Show the No Dependencies message when a Topology has no streams (#9413) ... Also changes the upstreams docs link	4 years ago
John Cowen	0ca54c608b	ui: Add new dev UI environment variables to the README (#9411)	4 years ago
John Cowen	4404b4f44c	ui: Set overflow: auto on the tab nav (#9402) ... This ensures scroll bars are mainly hidden, yet visible on small screens if the tabs are large enough for there to be an overflow	4 years ago
John Cowen	d602b303e1	ui: Rearrange Service detail page to load Topology and Routing tabs separately (#9401)	4 years ago
Kenia	a919b60f5c	ui: Return empty state for no health check in Topology Cards (#9403)	4 years ago
John Cowen	e921b3cf9a	ui: Install ember-intl (#9399) ... * ui: Install ember-intl Also: 1. Removes our own format-number in order to use ember-intl's instead 2. Moves format-time to format-short-time so as to not clash with ember-intls own format-time	4 years ago
John Cowen	5e150d7f04	ui: Lint get-environment.js (#9400)	4 years ago
John Cowen	9f07871971	ui: Use InformedAction for intention confirmation dialogs (#9398)	4 years ago
John Cowen	f111d6b3e3	ui: Dev/Test environment configurable metrics (#9345) ... In order to test certain setups for our metrics visualizations we need to be able to setup several different `ui_config` settings during development/testing. Generally in the UI, we use the Web Inspector to set various cookie values to configure the UI how we need to see it whilst developing, so this PR: 1. Routes `ui_config` through a dev time only `CONSUL_UI_CONFIG` env variable so we can change it via cookies vars. 2. Adds `CONSUL_METRICS_PROXY_ENABLE`, `CONSUL_METRICS_PROVIDER` and `CONSUL_SERVICE_DASHBOARD_URL` so it's easy to set/unset these only values during development. 3. Adds an acceptance testing step so we can setup `ui_config` to whatever we want during testing. 4. Adds an async 'repository-like' method to the `UiConfig` Service so it feels like a repository - incase we ever need to get this via an HTTP API+blocking query. 5. Vaguely unrelated: we allow cookie values to be set via the location.hash whilst in development only e.g. `/ui/services#CONSUL_METRICS_PROXY_ENABLE=1` so we can link to different setups if we ever need to. All values added here are empty/falsey by default, so in order to see how it was previously you'll need to set the appropriate cookies values, but you can now also easily preview/test the the metrics viz in different/disabled states (with differing `ui_config`)	4 years ago
John Cowen	14d043e5f8	ui: Show local datacenter by default on first visit (#9377) ... * Add `Local` property to Datacenters * If you have not previous datacenter, redirect the user to the local dc * Add an `is-local` class to the local datacenter in the DC picker	4 years ago
John Cowen	d6f27c6564	ui: Move linting to the `node:test` script (#9385) ... * Reconfigure linting to be a node-test * Fixup linting across the project	4 years ago
John Cowen	4e419b9b37	ui: Add External Sources Filter to Node > Service Instances (#9368) ... * Add service collections to get all ExternalServices * Add a basic collection helper * Use the collections to get all ExternalSources * Remove old Controllers	4 years ago
John Cowen	514270a413	ui: Controller dead code removal (#9367) ... * ui: Controller dead code removal This commit removes a little code that became 'dead' as a result of previous PRs/commits * Remove a little more from the settings Controller * Remove CSS related to being able to set the dashboard_url in settings	4 years ago
John Cowen	0f7c909f13	ui: Improved Discovery Chain resizing (#9360) ... * install on-resize modifier * Rerrange things to use on-resize modifier for positioning	4 years ago
John Cowen	21e5a8f0f3	ui: Remove random undefined from codemod run (#9370)	4 years ago
John Cowen	9d81319074	ui: Use pill/badge components for intention filter options (#9357)	4 years ago
John Cowen	2c3b6aec05	ui: Remove old style 'filterable' searching (#9356) ... * Switch upstream-instances to use new style of searchable * Add search action to DataCollection plus basic README * Use DataCollection for PowerSelect searching in child-selectors * Remove old style filterable search for role/policies and instances * Remove old helpers/components related to search/sort/filter	4 years ago
Kenia	a78566e2d8	ui: Send item name down to Stats to call endpoint (#9358) ... * Refactor Stats and Series components * Remove unused variable from being passed down to Stats * Add item name to be passed down to Stats	4 years ago
John Cowen	d6cb2b0d76	ui: Update UI README (#9346)	4 years ago
John Cowen	27c74f7141	ui: New search/sort/filtering bar for Node > ServiceInstances (#9326) ... * Model layer changes to turn Node:ServiceInstances into hasMany We tried to make something that feels a little like ember-data yet not leave our approach of re-shaping the JSON directly from the response. 1. We added transformHasManyResponse for re-shaping JSON for hasMany relationships. we avoided the normalize word as ember-data serialize methods usually return something JSON:API shaped and we distinctly don't want to do that. Transform was the best word we could think of. 2. The integration tests across all of our models here feel very much like those types of tests that aren't really testing much, or assert too much to an extent that they get in the way rather than be of any use. I'd very much like to move a lot of this to unit tests. Currently most of the fingerprinting functionality is unit tested and these integration tests were originally to give confidence that IDs and related properties were being added correctly. 3. We've added a hasMany relationship, but not the corresponding belongsTo - yet at least. We don't require the belongsTo right now, and if we do we can add it later. * Integrate ServiceInstance search bar for Node:ServiceInstances * Hide Node.Meta when on the Node:ServiceINstance page We use a little string replace hack here for a human-like label, this is soon to be replaced with proper i10n replacement * Always ensure that a Namespace is set, and add comment explaining	4 years ago
John Cowen	613be01f43	ui: Change URI helper to a template based approach (#9344) ... This moves our uri helper to use a the template renderer we already have for rendering URLs.	4 years ago
John Cowen	e0a4646768	ui: document-attrs helper (#9336) ... This commit adds a {{document-attrs}} helper, specifically for adding attributes to the root documentElement, which in our case is always <html>	4 years ago
Kenia	db5283ee24	ui: Refactor topology components (#9339) ... * Refactor Stats and Series components * Refactor metrics error message for ingress-gateway * Fix upLines icon positioning * Remove unused variable from being passed down to Stats	4 years ago
John Cowen	3be03029f6	ui: Native-ize class based helpers (#9337) ... * Install ember-decorators/observes * Remove filter-predicate * Nativize `env` and inject env service * Nativize all other class based helpers	4 years ago
John Cowen	9262c25fdf	ui: Lowercase the word 'instance' in Service listings (#9333)	4 years ago
John Cowen	4dfa7622d6	ui: Don't pluralize words when there is only 1 of the item (#9321)	4 years ago