237 Commits (ap/main-docs)

Author	SHA1	Message	Date
Kenia	b7a4fe0b4d	ui: Hide all metrics for ingress gateway services (#10858)	3 years ago
Kenia	b6cb12be96	ui: Add tests for topology metrics stats (#10600)	3 years ago
Kenia	88d8b84614	ui: Add support in Topology view for Routing Configurations (#10872)	3 years ago
Kenia	5a1b176d10	ui: Create Routing Configurations route and page (#10835)	3 years ago
John Cowen	9d18b08da3	ui: [BUGFIX] Add missing `@` to fix missing non-subset Failovers (#10913) ... This commit fixes a problem where parent Failovers where not showing (subset children were fine). Seems to have been introduced with a move/glimmer upgrade here #9154 so I'm adding a 1.9.x backport.	3 years ago
Kenia	f43eab3873	ui: Disabling policy form fields from users with 'read' permissions (#10902)	3 years ago
John Cowen	45ac42e0fe	ui: Unskip auth-method serializer test (#10878) ... During #9617 we added a list view only for AuthMethods, but not a detail view. We did add the Adapter/Serializer that collected/reshaped data for a detail view. The test for this serializer was skipped here, but I'm not sure why. We then added #9845 which began to use this AuthMethod Serializer, but we didn't go back to finish up the skipped test here either. This PR unskips this test and finishes off the test correctly.	3 years ago
John Cowen	3012cd6fa3	ui: [BUGFIX] Properly encode non-URL safe characters in OIDC responses (#10901) ... This commit fixes 2 problems with our OIDC flow in the UI, the first is straightforwards, the second is relatively more in depth: 1: A typo (1.10.1 only) During #10503 we injected our settings service into the our oidc-provider service, there are some comments in the PR as to the whys and wherefores for this change (https://github.com/hashicorp/consul/pull/10503/files#diff-aa2ffda6d0a966ba631c079fa3a5f60a2a1bdc7eed5b3a98ee7b5b682f1cb4c3R28) Fixing the typo so it was no longer looking for an unknown service (repository/settings > settings) fixed this. 2: URL encoding (1.9.x, 1.10.x) TL;DR: /oidc/authorize/provider/with/slashes/code/with/slashes/status/with/slashes should be /oidc/authorize/provider%2Fwith%2Fslashes/code%2Fwith%2Fslashes/status%2Fwith%2Fslashes When we receive our authorization response back from the OIDC 3rd party, we POST the code and status data from that response back to consul via acallback as part of the OIDC flow. From what I remember back when this feature was originally added, the method is a POST request to avoid folks putting secret-like things into API requests/URLs/query params that are more likely to be visible to the human eye, and POSTing is expected behaviour. Additionally, in the UI we identify all external resources using unique resource identifiers. Our OIDC flow uses these resources and their identifiers to perform the OIDC flow using a declarative state machine. If any information in these identifiers uses non-URL-safe characters then these characters require URL encoding and we added a helper a while back to specifically help us to do this once we started using this for things that required URL encoding. The final fix here make sure that we URL encode code and status before using them with one of our unique resource identifiers, just like we do with the majority of other places where we use these identifiers.	3 years ago
Kenia	54399c5394	ui: Update intention permissions notice wording (#10836)	3 years ago
Kenia	c41b6189e8	ui: Split up the socket mode from the socket path (#10581)	3 years ago
Kenia	5c8f1e6647	ui: Fix Health Checks in K/V form Lock Sessions Info section (#10767)	3 years ago
Kenia	ae34d7b4cd	ui: Fix dropdown option duplications (#10706)	3 years ago
John Cowen	e92b9e8e82	ui: [BUGFIX] Ensure we use the ns query param name when requesting permissions (#10608) ... Previously when namespaces were enabled, we weren't requesting permission for the actively selected namespace, and instead always checking the permissions for the default namespace. This commit ensures we request permissions for the actively selected namespace.	3 years ago
John Cowen	15f6b10e4a	ui: [BUGFIX] Ensure in-folder KVs are created in the correct folder (#10569) ... When clicking to create a KV within folder name, would would be viewing a form that was a form for creating a KV in the root, which when the user clicked to save, saved the KV in the root. For the moment at least I've removed the code that strips double slashes, and whilst this isn't ideal, it looks like we've picked up one of those bugs that turns into a 'feature', and completely reworking KV to not rely on the double slashes is not really an option right now.	3 years ago
John Cowen	70f29c2312	ui: [BUGFIX] Fix KV Code Editor syntax loading (#10605) ... This commit adds a bit of string wrangling to avoid the keys in our javascript source file also being transformed. Additionally, whilst looking at this we decided that Maps are a better dictionary than javascript objects, so we moved to use those here also (but this doesn't affect the issue)	3 years ago
John Cowen	6a0d4358e6	ui: Show the correct 'ACLs Disabled' page when ACLs are disabled (#10604) ... Adds 'can access ACLs' which means one of two things 1. When ACLs are disabled I can access the 'please enable ACLs' page 2. When ACLs are enabled, its the same as canRead	3 years ago
Daniel Nephin	fe76dc7068	Merge pull request #10552 from hashicorp/dnephin/ca-remove-rotation-period ... ca: remove unused RotationPeriod field	3 years ago
John Cowen	75c7491224	ui: Don't default to the default namespace, use the token default namespace instead (#10503) ... The default namespace, and the tokens default namespace (or its origin namespace) is slightly more complicated than other things we deal with in the UI, there's plenty of info/docs on this that I've added in this PR. Previously: When a namespace was not specified in the URL, we used to default to the default namespace. When you logged in using a token we automatically forward you the namespace URL that your token originates from, so you are then using the namespace for your token by default. You can of course then edit the URL to remove the namespace portion, or perhaps revisit the UI at the root path with you token already set. In these latter cases we would show you information from the default namespace. So if you had no namespace segment/portion in the URL, we would assume default, perform actions against the default namespace and highlight the default namespace in the namespace selector menu. If you wanted to perform actions in your tokens origin namespace you would have to manually select it from the namespace selector menu. This PR: Now, when you have no namespace segment/portion in the URL, we use the token's origin namespace instead (and if you don't have a token, we then use the default namespace like it was previously) Notes/thoughts: I originally thought we were showing an incorrectly selected namespace in the namespace selector, but it also matched up with what we were doing with the API, so it was in fact correct. The issue was more that we weren't selecting the origin namespace of the token for the user when a namespace segment was omitted from the URL. Seeing as we automatically forward you to the tokens origin namespace when you log in, and we were correctly showing the namespace we were acting on when you had no namespace segment in the URL (in the previous case default), I'm not entirely sure how much of an issue this actually was. This characteristic of namespace+token+namespace is a little weird and its easy to miss a subtlety or two so I tried to add some documentation in here for future me/someone else (including some in depth code comment around one of the API endpoints where this is very subtle and very hard to miss). I'm not the greatest at words, so would be great to get some edits there if it doesn't seem clear to folks. The fact that we used to save your previous datacenter and namespace into local storage for reasons also meant the interaction here was slightly more complicated than it needed to be, so whilst we were here we rejigged things slightly to satisfy said reasons still but not use local storage (we try and grab the info from higher up). A lot of the related code here is from before we had our Routlets which I think could probably make all of this a lot less complicated, but I didn't want to do a wholesale replacement in this PR, we can save that for a separate PR on its own at some point.	3 years ago
John Cowen	7550bb2c65	ui: Add intl debug helpers (#10513) (#10561) ... This commit adds a couple of debug utilities to help us to continue slowly adding i18n support: - We've added a CONSUL_INTL_DEBUG env/cookie variable to turn off variable interpolation within the t helper so you can see which variables are being interpolated. - We've added a CONSUL_INTL_LOCALE env/cookie which currently supports two 'pseudo-locales' - la-fk (fake latin) and - (just dashes) either of which will make it easier to see what has not been localized until we can add prettier rules to prevent adding any copy into templates at all. I would guess if we ever translated the app we would use this for looking at things whilst developing also - but as yet I've not adding anything for that here seeing as we don't translate anything. Both variables are dev-time only and all code for this is removed from the production build.	3 years ago
John Cowen	bd0dfc31b3	ui: CopyButton amends (#10511) ... * ui: Add with-copyable modifier * Use with-copyable modifier for our own CopyButton * Move copy-button styling and remove most of `copy-btn`	3 years ago
John Cowen	1d5e17d3cc	ui: Allow disabling of sourcemaps via env var (#10491)	3 years ago
John Cowen	db4ba43398	ui: Fixup definition-table + copy-button margin (#10512)	3 years ago
John Cowen	041a8cb293	ui: Enable specifying additional docfy config as json (#10464)	3 years ago
John Cowen	adb3223d56	ui: Colocate remaining CSS components to the app/components folder (#10328)	3 years ago
John Cowen	d8abd0b6e9	ui: Rename icons for consistency and remove unused icons (#10311) ... * ui: Standardize logo naming According to structure it should always be logo-name not name-logo * Make sure all our logos use logo-name format * Upgrade to @hashicorp/structure-icons 1.9.0 * Add `-color` to be consistent with other logos * Add ms logo back in * Remove all the old `*-color` icons from before when we got masks * Add missing files * Missed glimmer extend name change	3 years ago
John Cowen	5f51ac931b	ui: Increase z-index of main menus to avoid layering issues (#10428)	3 years ago
John Cowen	9bfa9afb5a	ui: Move all our icons to use CSS custom properties instead of SASS vars (#10298) ... * ui: Move all our icons to use CSS custom properties The good thing about SASS vars is, if you don't use them they get removed from the final build. Whereas with CSS we have no tree-shaking to get rid of unused CSS custom properties. We can mostly work around this and for some things like colors its no big deal if we have some hex-codes in the build that we don't use as hex-codes are relatively small. We've been slowly but surely moving all of our colors (and other things) to use CSS custom properties instead of SASS vars now that we have them available. This commit makes use of the 'tree-shaking' abilities of @extend to ensure that we only compile in the icons that we use. This commit is mostly churn-less as we already use @extend for the majority of our icons, so generally there is zero change here for working on the UI, but I did spot one single place where we were using SASS vars instead of @extend. This now uses the new form (second commit) Interestingly this reduces our CSS payload by ~2kb to ~53kb (around 25kb of that is these icons)	3 years ago
John Cowen	2f9e968d38	ui: %horizontal-kv-list CSS component (and related) (#10285) ... This commit uses docfy to isolate the individual parts and options and investigates the why you might use certain options and document how you might use certain options. Originally we used a single %icon-definition CSS component to represent this, but seeing as some of them don't have icons, it didn't seem like the best name. So this PR splits this component into various different ones and then uses the new ones to continue to provide a now deprecated %icon-definition. The component is currently a CSS only component that assumes a single (or multiple) description lists for its markup component, and provides for multiple different options (including a reversed mode which I'm still not totally sure about, but we don't use this right now anyway). - %icon-definition - %horizontal-kv-list - %csv-list - %tag-list - %badge	3 years ago
John Cowen	8fb3ef99c8	ui: Move intention description field (#10292) ... ...from under the permissions widget to over the permissions widget	4 years ago
John Cowen	5518c36e62	ui: Fixup prettier for scss files and run (#10296) ... * ui: Ensure prettier makes all our scss files pretty * Run prettier on all our scss files	4 years ago
Kenia	8a34582273	ui: Create and use collapsible notices component (#10270) ... * Create and use collapsible notices * Refactor collapsible-notices * Split up the topology acceptance tests * Add acceptance tests for tproxy notices * Add component file * Adds additional TProxy notices tests * Adds conditional to only show collapsable if more than 2 notices are present * Adds changelog * Refactorting the conditonal for collapsing the notices * Renaming undefinedIntention to be notDefinedIntention * Refactor tests	4 years ago
John Cowen	206e414908	ui: Unix Domain Socket support (#10287) ... This commit adds UI support for Unix Domain Sockets for upstream and downstreams (see #9981 and #10252)	4 years ago
John Cowen	dd280ee2b2	ui: Support Route optional parameters/segments (#10212) ... Moves our URLs with 'optional namespace segment' into a separately abstracted 'optional URL segment' feature	4 years ago
John Cowen	56a165b489	ui: Colocate old base components into our app/component folder (#10275)	4 years ago
John Cowen	7731aab820	ui: Minor CSS tweaks (#10295)	4 years ago
John Cowen	f6605b2770	ui: Add docs for AppView (#10265) ... * ui: change coloring of secondary navigation elements * Remove top border, this was probably from older designs/iterations * ui: Move app-view styles into components also... 1. Remove dead %app-view-content-error 2. Remove TabNav border overwriting * Bring into line with our 'project standard' class/attributes pattern * Add docs for AppView	4 years ago
John Cowen	be13c4ff42	ui: change coloring of secondary navigation elements (#10259)	4 years ago
John Cowen	f69c4a2501	ui: Miscellaneous Lock Session fixes (#10225)	4 years ago
John Cowen	eb396b0409	ui: Bump node to v14 (#10238)	4 years ago
John Cowen	981b8af307	ui: Use babel plugin to strip calls to runInDebug (#10237)	4 years ago
John Cowen	1fe17b720a	ui: Serf Health Check warning notice (#10194) ... When the Consul serf health check is failing, this means that the health checks registered with the agent may no longer be correct. Therefore we show a notice to the user when we detect that the serf health check is failing both for the health check listing for nodes and for service instances. There were a few little things we fixed up whilst we were here: - We use our @replace decorator to replace an empty Type with serf in the model. - We noticed that ServiceTags can be null, so we replace that with an empty array. - We added docs for both our Notice component and the Consul::HealthCheck::List component. Notice now defaults to @type=info.	4 years ago
Kenia	062590180f	ui: Add conditionals to Lock Session list items (#10121) ... * Add conditionals to Lock Session list items * Add changelog * Show ID in details if there is a name to go in title * Add copy-button if ID is in the title * Update TTL conditional * Update .changelog/10121.txt Co-authored-by: John Cowen <johncowen@users.noreply.github.com> Co-authored-by: John Cowen <johncowen@users.noreply.github.com>	4 years ago
John Cowen	b93df0e22d	ui: Fix nspace spacing issue (#10157) ... This fixes the spacing bug in nspaces only by only showing Description if the namespace has one, and removing the extra 2 pixel margin of dds for when dts aren't rendered/don't exist.	4 years ago
John Cowen	8522c83283	ui: Adds ability to show a 'partial' list in list-collections (#10174) ... * ui: Add support for showing partial lists in ListCollection * Add CSS for partial 'View more' button, and move all CSS to /components * Enable partial view for intention permissions	4 years ago
John Cowen	4c983cfd3e	ui: Remove top border from Consul::AuthMethod::List (#10195)	4 years ago
John Cowen	f61866fda6	ui: Loader amends/improvements (#10181) ... * ui: Loader amends/improvements 1. Create a JS compatible template only 'glimmer' component so we can use it with or without glimmer. 2. Add a set of `rose` colors. 3. Animate the brand loader to keep it centered when the side navigation appears. 4. Tweak the color of Consul::Loader to use a 'rose' color. 5. Move everything loader related to the `app/components/` folder and add docs.	4 years ago
John Cowen	16eb3263a3	ui: [BUGFIX] De-duplicate Tag rendering (#10186) ... * Add some tests for duplicated and non-duplicated tags * Ensure tags get de-duped and add docs * Update docs to include info on the recursive-ness	4 years ago
Mike Morris	a80a1d87bf	ui: add data-enterprise-logo to logo svg for enterprise	4 years ago
Hector Simpson	a1cb79b1f8	Update brand assets (#10081) ... * Update header logo and inline icon * Update full logos + layout on loading screen * Update favicon assets and strategy - Switches to serve an ico file alongside an SVG file - Introduces an apple-touch-icon * Removes unused favicon/meta assets * Changelog item for ui * Create component for logo * Simplify logo component, set brand color * Fix docs loading state CSS issue	4 years ago
Kenia	f35af2bb59	ui: Add TProxy Mode notice banner to service instance Upstreams tab (#10136)	4 years ago