794 Commits (49d3e40d81a085153414a723bcb7b365242dad20)

Author	SHA1	Message	Date
John Cowen	5ab7e48862	ui: Native CSS Icon Composition (#12461) ... This commit/PR beings to move away from using CSS preprocessing for our icons and towards using native CSS via native CSS property composition	3 years ago
John Cowen	ee0e8bf500	ui: App-ify Lock Sessions (#12482) ... This commit moves our in-app LockSessions code into an external 'app', which can theoretically be side-loaded but for now it just makes for good isolation/code hygiene. Functionally, there is kind of one change here, and that is we only show the 'Lock Session' tab if you have permissions to see them. Currently as our UI authorization endpoint needs to be changed slightly to suit our usecase, you will always have permissions to see Lock Sessions as we hardcode the session:read to true (obvs this is a frontend thing, not a backend thing), so it doesn't really change anything from a user perspective. Also added very bare docs while I was here. Small note here, ideally we need to add the each individual tab depending on whether an 'app' is enabled or not instead of just permissions, ideally it would be done totally from The Outside rather than a can based conditional on the inside, just something else to be thinking about for the future.	3 years ago
John Cowen	2168a44413	ui: Add CustomElement Transform (#12540) ... This PR adds an Ember-Glimmer/Handlebars transform to drop the documentation copy from the CustomElement component out of the build, therefore it is not bundled into the code either at development time or production build time.	3 years ago
John Cowen	0e122479fa	ui: Add license endpoint/datasource (#12506) ... * ui: Add auto-pilot/state endpoint usage (merged into DC models) (#12514) * ui: Catalog Health Overview DataSource (#12520)	3 years ago
John Cowen	ec536340df	ui: Remove Settings Ember Route (#12488)	3 years ago
John Cowen	55851c784f	ui: DistributionMeter Component (#12452)	3 years ago
Evan Rowe	b93401795a	Merge pull request #12400 from chinmaym07/ui/feature/add-tags-tab-to-gateways ... ui: [Bugfix] Added Tags tab to gateways(just like exists for non-gateway services)	3 years ago
John Cowen	7a1c3a8c60	ui: List Primary and Local DCs first in the Datacenter selector (#12478)	3 years ago
John Cowen	bac6c00906	ui: Fix loading icon size (adds a viewbox) (#12479)	3 years ago
John Cowen	25f3ebd66a	ui: CustomElement component (#12451) ... Builds on attach-shadow, adopt-styles and ShadowTemplate, this commit adds ShadowHost and finally CustomElement. CustomElement is a renderless component to help with the creation of native HTML Custom Elements along with runtime type checking and self-documentation for attributes, slots, cssprops and cssparts. As you will probably see there is a little more work to come here. But in the same breath, everything would be fine to go in as is.	3 years ago
John Cowen	9289c9b491	ui: Add docs for `<Action />` component (#12502)	3 years ago
John Cowen	121bd2e0ab	ui: PagedCollection component (#12404) ... * ui: PagedCollection component * ui: Use PagedCollection (#12436) * ui: Integrate PagedCollection into DisclosureMenu * Integrate PageCollection into DC, Nspace and Partition menus	3 years ago
John Cowen	79a07c7a3d	ui: Add new component blueprint (#12421) ... This adds a new component blueprint for all our components. We've been using README.mdx files for quite some time to document our components for other engineers. This adds a default file to help new engineers get started writing useful documentation. These README.mdx file are also very useful for building out components in isolation from scratch and and some point will be used to run automated component testing.	3 years ago
John Cowen	73b6687c5b	ui: Transition App Chrome to use new Disclosure Menus (#12334) ... * Add %panel CSS component * Deprecate old menu-panel component * Various smallish tweaks to disclosure-menu * Move all menus in the app chrome to use new DisclosureMenu * Follow up CSS to move all app chrome menus to new components * Don't prevent default any events from anchors * Add a tick to click steps	3 years ago
chinmaym07	bbf4b8b759	fix: Added Tags tab to gateways(just like exists for non-gateway services) ... Signed-off-by: chinmaym07 <b418020@iiit-bh.ac.in>	3 years ago
John Cowen	360cf89ea5	ui: Fixup displaying a Nspace default policy when expanding the preview pane (#12316)	3 years ago
John Cowen	717621698d	ui: Replace CollapsibleNotices with more a11y focussed Disclosure component (#12305) ... * Delete collapsible notices component and related helper * Add relative t action/helper to our Route component * Replace single use CollapsibleNotices with multi-use Disclosure	3 years ago
John Cowen	9b22300c59	ui: Start using mermaid state diagrams in our docs (#12350)	3 years ago
Florian Apolloner	f01f00fc84	Support for connect native services in topology view. (#12098)	3 years ago
John Cowen	0e730328bd	ui: Disclosure Component amends plus DisclosureMenu Component (#12304) ... * ui: Disclosure amends plus DisclosureMenu Co-authored-by: Jamie White <jamie@jgwhite.co.uk>	3 years ago
John Cowen	0ac32122f1	ui: Make sure saving intentions from topology includes the partition (#12317)	3 years ago
John Cowen	cf98691e85	ui: Stop ember-data overwriting SyncTimes (#12315)	3 years ago
John Cowen	c1b4e33dee	ui: Exclude Service Health from Node listing page (#12248) ... This commit excludes the health of any service instances from the Node Listing page. This means that if you are viewing the Node listing page you will only see failing nodes if there are any Node Checks failing, Service Instance Health checks are no longer taken into account. Co-authored-by: Jamie White <jamie@jgwhite.co.uk>	3 years ago
John Cowen	d49ee8e355	ui: Ensure proxy instance health is taken into account in Service Instance Listings (#12279) ... We noticed that the Service Instance listing on both Node and Service views where not taking into account proxy instance health. This fixes that up so that the small health check information in each Service Instance row includes the proxy instances health checks when displaying Service Instance health (afterall if the proxy instance is unhealthy then so is the service instance that it should be proxying) * Refactor Consul::InstanceChecks with docs * Add to-hash helper, which will return an object keyed by a prop * Stop using/relying on ember-data type things, just use a hash lookup * For the moment add an equivalent "just give me proxies" model prop * Start stitching things together, this one requires an extra HTTP request ..previously we weren't even requesting proxies instances here * Finish up the stitching * Document Consul::ServiceInstance::List while I'm here * Fix up navigation mocks Name > Service	3 years ago
John Cowen	ed5204b6b5	ui: ShadowTemplate component (#12259)	3 years ago
John Cowen	b179f9fa91	ui: aria-menu modifier (#12262) ... aria-menu modifier plus Menu component (#12266)	3 years ago
John Cowen	69596a6433	ui: Disclosure Component (#12263)	3 years ago
John Cowen	39f15306d9	ui: Change approach to loading debug.css (#12242) ... We need a way to load certain CSS based on the environment you are viewing, i.e. we have debug CSS that we use for our Eng Documentation and various other DX utilities that shouldn't be compiled into our production or test builds. Previously we would compile two entirely different CSS files (app and debug) and the load one or the other depending on which environment you were in. This approach just empties out the debug.css file in certain environments (prod/test) which means we can just import that file from app. When in staging/development this imports the contents of debug.css (quite a bit of CSS) whereas when building for production/test this debug.css is emptied out during the build process. There is a slight little hack in order to have this work, we import _debug.scss which imports the debug.scss file. I couldn't for the life of me figure out how to have broccoli empty out a file during the build process, so instead we essentially copy over debug.scss during dev and create an empty file during prod to _debug.scss. When using make build to build an artifact for production CSS remains at ~58kb (during dev its a lot bigger than this)	3 years ago
John Cowen	0f94ce3964	ui: Alias all our Structure Icons to Flight Icons (#12209)	3 years ago
John Cowen	417cb8d838	ui: attach-shadow modifier (#12207) ... * ui: attach-shadow modifier * ui: adopt-styles helper (#12208)	3 years ago
John Cowen	189895e8fa	ui: style-map helper (#12203)	3 years ago
John Cowen	59756e4a88	ui: on-outside modifier (#12206)	3 years ago
John Cowen	de13e207e5	ui: css-prop modifier (#12205) ... Get the value for a single specific CSS Property from the modified element. returns can be specified either as a second parameter or an option.	3 years ago
John Cowen	eac0f5b860	ui: class-map helper (#12202) ... {{class-map}} is used to easily add a list of classes, conditionally, and have them all formatted nicely ready to be printed in a DOM class attribute. For ease, as well as using entries, you can also just provide a simple string without the boolean and that class will always be added.	3 years ago
John Cowen	e6a084807b	ui: Enable theming (#12134) ... plus Themeable icons (#12135)	3 years ago
John Cowen	61d7acd51f	ui: Tweak some code related meta information (#12117) ... * ui: Correct some meta info * Encoder doesn't take an argument whereas decoder does * Math.trunc looks like the closest to parseInt but using the correct type * use a dynamic string when setting things on window	3 years ago
Dan Upton	ca3aca92c4	[OSS] Remove remaining references to master (#11827)	3 years ago
John Cowen	48ace89c6a	ui: Runtime Injectable Components (#11969) ... - Simplifies how we 'import' our configuration files a little in order to make them more grokable. - Starts to exclude files based on explicit configuration rather than convention. - Adds the first instance of us being able to select an implementation (of multiple) of a component at runtime.	3 years ago
John Cowen	cdb8a35501	ui: Fixup KV folder creation then further creation within that folder (#12081) ... The fix here is two fold: - We shouldn't be providing the DataSource (which loads the data) with an id when we are creating from within a folder (in the buggy code we are providing the parentKey of the new KV you are creating) - Being able to provide an empty id to the DataSource/KV repository and that repository responding with a newly created object is more towards the "new way of doing forms", therefore the corresponding code to return a newly created ember-data object. As we changed the actual bug in point 1 here, we need to make sure the repository responds with an empty object when the request id is empty.	3 years ago
John Cowen	9192452885	ui: Adds a notice for non-primary intention creation (#11985)	3 years ago
John Cowen	b50c7cbef9	ui: Fix up wiring or empty state login button (#11981) ... * ui: Some ACL component documentation (#11982)	3 years ago
John Cowen	0ac27ef473	ui: Alter position of dashboard button in the service instance header (#11988)	3 years ago
John Cowen	e424e3dc83	ui: Allow templateName paths to be relative (#11955)	3 years ago
John Cowen	2aaa96bf96	ui: First pass at writing some data layer related Eng docs (#11203)	3 years ago
Kenia	aaace559e2	ui: Adding Partition to topology card (#11805)	3 years ago
John Cowen	78e9c0d2d9	ui: Ensure the partition is passed through to the request for the SSO auth URL (#11979) ... * Make sure the mocks reflect the requested partition/namespace * Ensure partition is passed through to the HTTP adapter * Pass AuthMethod object through to TokenSource in order to use Partition * Change up docs and add potential improvements for future * Pass the query partition back onto the response * Make sure the OIDC callback mock returns a Partition * Enable OIDC provider mock overwriting during acceptance testing * Make sure we can enable partitions and SSO post bootup only required ...for now * Wire up oidc provider mocking * Add SSO full auth flow acceptance tests	3 years ago
John Cowen	d9a315e2a5	ui: Remove KV pre-flight auth check (#11968) ... * ui: Don't even ask whether we are authorized for a KV... ...just let the actual API tell us in the response, thin-client style. * Add some similar commenting for previous PRs related to this problem	3 years ago
John Cowen	6c240fbf2d	ui: Ensure service instance data does not get re-written on blocking refresh (#11903) ... * Add some less fake API data * Rename the models class so as to not be confused with JS Proxies * Rearrange routlets slightly and add some initial outletFor tests * Move away from a MeshChecks computed property and just use a helper * Just use ServiceChecks for healthiness filtering for the moment * Make TProxy cookie configurable * Amend exposed paths and upstreams so they know about meta AND proxy * Slight bit of TaggedAddresses refactor while I was checking for `meta` etc * Document CONSUL_TPROXY_ENABLE	3 years ago
John Cowen	adb0fa38d2	ui: [BUGFIX] Fixes an issue when editing intentions from the service > intentions sub tab (#11937) ... We recently changed the intentions form to take a full model of a dc rather than just the string identifier (so {Name: 'dc', Primary: true} vs just 'dc' in order to know whether the DC is the primary or not. Unfortunately, we only did this on the global intentions page not the per service intentions page. This makes it impossible to save an intention from the per service intention page (whilst you can still save intentions from the global intention page as normal). The fix here pretty much copy/pastes the approach taken in the global intention edit template over to the per service intention edit template. Tests have been added for creation in the per service intention section, which again are pretty much just copied from the global one, unfortunately this didn't exist previously which would have helped prevent this.	3 years ago
John Cowen	6bdb2c2216	ui: Upgrade AuthDialog (#11913) ... - Move AuthDialog to use a Glimmer Component plus native named blocks/slots. - Unravel the Auth* contextual components, there wasn't a lot of point having them as contextual components and now the AuthDialog (non-view-specific state machine component) can be used entirely separately from the view-specific components (AuthForm and AuthProfile). - Move all the ACL related components that are in the main app chrome/navigation (our HashicorpConsul component) in our consul-acls sub package/module (which will eventually be loaded on demand only when ACLs are enabled)	3 years ago
John Cowen	1f8960d74b	ui: Fix dark borders on certain visualizations (#11959)	3 years ago
John Cowen	6d0a73c0eb	ui: Add selective no-console eslint rule (#11938)	3 years ago
John Cowen	31dee2340f	ui: Only allow partition creation with a single datacenter setup (#11817)	3 years ago
John Cowen	64767b53e7	ui: Configure routes in route config rather than classes (#11900)	3 years ago
James Tran	d1ac40e967	ui: Add XML syntax highlighting to key/value editor (#11785) ... * ui: Add XML syntax highlighting to key/value editor * ui: Make explicit options that are specific to XML for clarity	3 years ago
John Cowen	f8b336b9a0	ui: Fix URL params decoding (#11931) ... * ui: Move wildcard param decoding to routlet service	3 years ago
John Cowen	72e5d7e1d4	ui: Ensure disconnect error doesn't appear w/auth change on some pages (#11905)	3 years ago
John Cowen	3949acbc1d	ui: Remove any route level auth checks (#11891)	3 years ago
John Cowen	2f6ae96ccd	ui: Fixup names of Meta for instance search, also add Node (#11774)	3 years ago
John Cowen	2c1eb052e3	ui: Various empty state improvements/fixups (#11892) ... * ui: Add login button to per service intentions for zero results * Add login button and consistent header for when you have zero nodes * `services` doesn't exists use `items` consequently: Previous to this fix we would not show a more tailored message for when you empty result set was due to a user search rather than an empty result set straight from the backend * Fix `error` > `@error` in ErrorState plus code formatting and more docs * Changelog	3 years ago
John Cowen	72cddf4f49	ui: Check for `intention` ACL resources, not `intentions` ACL resources (#11880) ... * ui: Check for `intention` ACL resources, not `intentions` ACL resources * ui: Flip the test assertion to see the create button temporarily	3 years ago
John Cowen	ba70fe5e55	ui: Fix up missing policy delete warning modal (#11868) ... Fixes an issue where the policy detail page delete button would disappear when clicking rather than showing a modal confirmation	3 years ago
John Cowen	20cc028ac0	ui: Fixes an issue with the version footer when scrolling (#11850)	3 years ago
John Cowen	11ab84f840	ui: Add version information back into the footer (#11803)	3 years ago
John Cowen	2ba0e86d6d	ui: Disable setting wildcard partitions for intentions (#11804)	3 years ago
John Cowen	79b25901d7	ui: Change the URL prefix of partitions from `-` to `_` (#11801)	3 years ago
John Cowen	79679da840	ui: Fix a problem showing the default part in a non-primary (#11800) ... When switching to a non-primary datacenter we should only show the word 'default' in place of the partition menu, this fixes up a bug preventing that from happening due to erroneous if/let nesting	3 years ago
John Cowen	b5c0b35ef7	ui: Prefer shorter partition word in certain places vs Admin Partition (#11772)	3 years ago
John Cowen	be23aab001	ui: Ensure we show a special readonly page for intentions (#11767)	3 years ago
John Cowen	e4d33187b9	ui: reuse BucketList for intention view pages (#11765)	3 years ago
John Cowen	4400d7c071	ui: Change partitions to expect `[]` from the API (#11791)	3 years ago
John Cowen	c410d295be	ui: Amends to Routing visualization for partitions (#11747) ... * Update disco fixtures now we have partitions * Add virtual-admin-6 fixture with partition 'redirects' and failovers * Properly cope with extra partition segment for splitters and resolvers * Make 'redirects' and failovers look/act consistently * Fixup some unit tests	3 years ago
John Cowen	c434fefda2	ui: Fixup notifications for tokens using and topology intention saving (#11763)	3 years ago
John Cowen	340a0e03f5	ui: Make 'dangerous' buttons have white text even in dark theme (#11756)	3 years ago
John Cowen	05fabbe898	ui: Improve error messaging for when we can't make a slug (#11697) ... Ember Data requires the usage of unique ID to identify its records in the frontend, and we use a centralized function to do that for all records. There are occasions where it can't make an ID, usually this is a bug our side, but there are occasions where Consul might not be giving us the data needed to make an ID, for example if a Service comes down to us with a blank Name. Whilst this isn't a problem to be fixed in the UI, I thought we could make an improvement here by giving a little more info as to why the UI cannot make a unique ID. This is currently semi-hidden away in the javascript console, but we could potentially surface this in the UI itself as a larger task. I figured this smaller task could help folks in the meantime if they hit upon this as they might open up the javascript console themselves to see whats up and they'd at least get this extra clue.	3 years ago
John Cowen	c04701b387	ui: remove old head.hbs file we don't need (#11678)	3 years ago
John Cowen	77412fd2e0	ui: Stop tables overlapping with their headers when scrolling (#11670)	3 years ago
John Cowen	85c39092c0	ui: Adds basic support for partition exports to Service listings (#11702) ... Also: * ui: Add cross partition linking and rollout BucketList (#11712) * ui: Add exported service partition to the source filter menu (#11727)	3 years ago
John Cowen	171cb0a247	ui: Adds partition support to Service and Node Identity templates (#11696)	3 years ago
John Cowen	f27685cc40	ui: Adds basic support for the Routing tab viz with partitions (#11679)	3 years ago
John Cowen	b07ff88874	ui: Add documentation link for partitions (#11668)	3 years ago
John Cowen	4e0507aed3	ui: Don't cache event sources following a 401 (#11681)	3 years ago
John Cowen	f3d9565277	ui: Refactor KV and Lock Sessions following partitions update (#11666) ... This commit uses all our new ways of doing things to Lock Sessions and their interactions with KV and Nodes. This is mostly around are new under-the-hood things, but also I took the opportunity to upgrade some of the CSS to reuse some of our CSS utils that have been made over the past few months (%csv-list and %horizontal-kv-list). Also added (and worked on existing) documentation for Lock Session related components.	3 years ago
John Cowen	5ad82693d8	ui: Don't check global session permissions for a single session (#11657)	3 years ago
John Cowen	b5c8af4580	ui: Add `Service.Partition` as available variable for dashboard urls (#11654)	3 years ago
John Cowen	bda64f98b3	ui: Don't show policy datacenter selector for non-default partitions (#11656)	3 years ago
John Cowen	26341f0b05	ui: Move nspace CRUD to use the same approach as partitions (#11633) ... This sounds a bit 'backwards' as the end goal here is to add an improved UX to partitions, not namespaces. The reason for doing it this way is that Namespaces already has a type of 'improved UX' CRUD in that it has one to many relationship in the form when saving your namespaces (the end goal for partitions). In moving Namespaces to use the same approach as partitions we: - Ensure the new approach works with one-to-many forms. - Test the new approach without writing a single test (we already have a bunch of tests for namespaces which are now testing the approach used by both namespaces and partitions) Additionally: - Fixes issue with missing default nspace in the nspace selector - In doing when checking to see that things where consistent between the two, I found a few little minor problems with the Admin Partition CRUD so fixed those up here also. - Removed the old style Nspace notifications	3 years ago
John Cowen	f7b8df281b	ui: Add `Service.Namespace` variable to dashboard URL templates (#11640) ... We currently allow only Datacenter, Service.Name, this PR adds Service.Namespace.	3 years ago
John Cowen	3f131dcf34	ui: Notifications re-organization/re-style (#11577) ... - Moves where they appear up to the <App /> component. - Instead of a <Notification /> wrapping component to move whatever you use for a notification up to where they need to appear (via ember-cli-flash), we now use a {{notification}} modifier now we have modifiers. - Global notifications/flashes are no longer special styles of their own. You just use the {{notification}} modifier to hoist whatever component/element you want up to the top of the page. This means we can re-use our existing <Notice /> component for all our global UI notifications (this is the user visible change here)	3 years ago
John Cowen	2b8273bc6e	ui: Support for SSO with Admin Partitions (#11604) ... * Upgrade AuthForm and document current state a little better * Hoist SSO out of the AuthForm * Bare minimum admin partitioned SSO also: ui: Tabbed Login with Token or SSO interface (#11619) - I upgraded our super old, almost the first ember component I wrote, to use glimmer/almost template only. This should use slots/contextual components somehow, but thats a bigger upgrade so I didn't go that far. - I've been wanting to upgrade the shape of our StateChart component for a very long while now, here its very apparent that it would be much better to do this sooner rather than later. I left it as is for now, but there will be a PR coming soon with a slight reshaping of this component. - Added a did-upsert modifier which is a mix of did-insert/did-update - Documentation added/amended for all the new things.	3 years ago
John Cowen	b84ee47ff0	ui: Fix brand coloring for inline-code plus docs (#11578) ... * ui: Fix brand coloring for inline-code plus docs Also use --tones instead of --black/--white (#11601) Co-authored-by: Evan Rowe <ev.rowe@gmail.com>	3 years ago
John Cowen	392dedc3e2	ui: Upgrades token sourcing related components to Glimmer+docs (#11592)	3 years ago
John Cowen	b4a7278e72	ui: Add eng docs for our render-template helper (#11359) ... Adds Engineer documentation for our recently improved render-template helper, plus some ideas for further work/improvement here should anyone ever want to pick that up.	3 years ago
John Cowen	b7bd938411	ui: Store the default partition when logging in (#11591) ... Make sure we store the default Partition for a users token.	3 years ago
John Cowen	35ccc2bc29	ui: Adding partitions + icons to upstreams/upstream instances (#11556) ... Adds a partition badge/label/visual to upstreams and upstream instances.	3 years ago
John Cowen	2ef9ea00b1	ui: When certain menus are selected reset either nspace/partition (#11479) ... For our dc, nspace and partition 'bucket' menus, sometimes when selecting one 'bucket' we need to reset a different 'bucket' back to the one that your token has by default (or the default if not). For example when switching to a different partition whilst you are in a non-default namespace of another partition, we need to switch you to the token default namespace of the partition you are switching to.	3 years ago
John Cowen	28b1e642cb	ui: Upstream CSS Tweaks (#11554) ... Various CSS tweaks/HTML cleanup around upstreams (but impacts other 'rows') - Prefer {{tooltip}} to <Tooltip> - Removed some now unnecessary spans - Stop using an empty class="" for styling purposes. - Renamed any classes used to identify response properties to follow the exact property name but kebab-cased. - Fixed up the alignment of things in the rows when used with a 'tiny copy button' (see screengrab) which was minus positioning and knocking things out (pending a proper refactor of our copy button CSS which is from the very very start of things)	3 years ago
John Cowen	c847c9a8cb	ui: Fixup partiton > partition typo (#11572)	3 years ago
Alessandro De Blasis	53a61349e1	ui: Fix code editor resizing and restyle (#11474) ... Fixes an issue where the code editor would not resizing to the full extent of the browser window plus CodeEditor restyling/refactoring - :label named block - :tools named block - :content named block - code and CSS cleanup - CodeEditor.mdx Signed-off-by: Alessandro De Blasis <alex@deblasis.net> Co-authored-by: John Cowen <johncowen@users.noreply.github.com>	3 years ago
John Cowen	6557f52962	ui: Bump our browser support (#11505) ... Bumps our browser support to a "rough ~2 years back" approach.	3 years ago
John Cowen	66f1dd9425	ui: [Port] Ensure the tokens default nspace (and partition) is passed thru to the auth endpoint (#11490) ... Most HTTP API calls will use the default namespace of the calling token to additionally filter/select the data used for the response if one is not specified by the frontend. The internal permissions/authorize endpoint does not do this (you can ask for permissions from different namespaces in on request). Therefore this PR adds the tokens default namespace in the frontend only to our calls to the authorize endpoint. I tried to do it in a place that made it feel like it's getting added in the backend, i.e. in a place which was least likely to ever require changing or thinking about. Note: We are probably going to change this internal endpoint to also inspect the tokens default namespace on the backend. At which point we can revert this commit/PR. * Add the same support for the tokens default partition	3 years ago
Kenia	37de276350	ui: Add Consul API Gateway as an external source (#11371)	3 years ago
John Cowen	537c4f32f7	Revert "Merge pull request #11328 from radiantly/ui/feature/allow-${}-style-interpolation" ... This reverts commit cd55c0cda3, reversing changes made to 14af8cb7a9.	3 years ago
John Cowen	aeaa0a6c96	ui: Temporarily force all UI capabilities (#11520) ... Temporarily revert to pre-1.10 UI functionality by overwriting frontend permissions. These are used to hide certain UI elements, but they are still enforced on the backend. This temporary measure should be removed again once https://github.com/hashicorp/consul/issues/11098 has been resolved	3 years ago
John Cowen	1515c3e667	ui: Filter global intentions list by selected partition (#11475) ... * ui: Filter global intentions list by namespace and partition Filters global intention listing by the current partition rather than trying to use a wildcard.	3 years ago
Kenia	4c2fa322a1	ui: Topology view with no dependencies (#11280)	3 years ago
John Cowen	25961355c1	ui: Only show partition / partition selector if enabled (#11484) ... I missed that partitions should never display if you can't use them, not just if you can't choose them.	3 years ago
John Cowen	72be2dd204	ui: Ensure we check intention service prefix permissions for per service (#11409) ... Port of: Ensure we check intention service prefix permissions for per service (#11270) Previously, when showing some action buttons for 'per service intentions' we used a global 'can I do something with any intention' permission to decide whether to show a certain button or not. If a user has a token that does not have 'global' intention permissions, but does have intention permissions on one or more specific services (for example via service / service_prefix), this meant that we did not show them certain buttons required to create/edit the intentions for this specific service. This PR adds that extra permissions check so we now check the intentions permissions per service instead of using the 'global' "can I edit intentions" question/request. **Notes:** - If a HTML button is `disabled` this means tippy.js doesn't adopt the popover properly and subsequently hide it from the user, so aswell as just disabling the button so you can't active the popover, we also don't even put the popover on the page - If `ability.item` or `ability.item.Resources` are empty then assume no access **We don't try to disable service > right hand side intention actions here** Whether you can create intentions for a service depends on the _destination_ of the intention you would like to create. For the topology view going from the LHS to the center, this is straightforwards as we only need to know the permissions for the central service, as when you are going from the LHS to the center, the center is the _destination_. When going from the center to the RHS the _destination[s]_ are on the RHS. This means we need to know the permissions for potentially 1000s of services all in one go in order to know when to show a button or not. We can't realistically discover the permissions for service > RHS services as we'd have either make a HTTP request per right hand service, or potentially make an incredibly large POST request for all the potentially 1000s of services on the right hand side (more preferable to 1000s of HTTP requests). Therefore for the moment at least we keep the old functionality (thin client) for the middle to RHS here. If you do go to click on the button and you don't have permissions to update the intention you will still not be able to update it, only you won't know this until you click the button (at which point you'll get a UI visible 403 error) Note: We reversed the conditional here between 1.10 and 1.11 So this make 100% sense that the port is different here to 1.11	3 years ago
Luke Kysow	c61208146d	Use "Open dashboard" everywhere (#11406) ... Previously we had "Open metrics Dashboard" and "Configure metrics dashboard" in the topology cards and then we had "Open Dashboard" in the top nav when the dashboard was configured. Now we use "Open dashboard" and "Configure dashboard". This change was made for consistency in wording and casing. In addition, the dashboard could be used for metrics but also other dashboards so there's no need to scope it only to metrics. Also the config is: ```hcl ui_config { dashboard_url_templates } ``` Which does not mention metrics	3 years ago
John Cowen	76f5de1455	ui: Ensure dc selector correctly shows the currently selected dc (#11380) ... * ui: Ensure dc selector correctly shows the currently selected dc * ui: Restrict access to non-default partitions in non-primaries (#11420) This PR restricts access via the UI to only the default partition when in a non-primary datacenter i.e. you can only have multiple (non-default) partitions in the primary datacenter.	3 years ago
John Cowen	b96794401f	ui: Add initial "How 2 Test UI" docs (#11296) ... Attempt to document out what a beginner to the project needs to know here in order to get started quickly Co-authored-by: Jared Kirschner <85913323+jkirschner-hashicorp@users.noreply.github.com>	3 years ago
John Cowen	86c88cc43a	ui: Ensure we provide an id for filtering policy-tokens (#11311)	3 years ago
Chris S. Kim	76bbeb3baf	ui: Pass primary dc through to uiserver (#11317) ... Co-authored-by: John Cowen <johncowen@users.noreply.github.com>	3 years ago
Jared Kirschner	cd55c0cda3	Merge pull request #11328 from radiantly/ui/feature/allow-${}-style-interpolation ... ui: Allow ${ } interpolation for UI Dashboard template URLs	3 years ago
John Cowen	e938e0afe0	ui: Remove remaining partition FIXME comments (#11312)	3 years ago
cooleditphoto	f801662f5c	ui: Topology metrics view outbound bytes color blue (#11343)	3 years ago
radiantly	fd51b3e76e	ui: Allow ${} interpolation for template URLs	3 years ago
John Cowen	a1a0ce1dad	ci: Only install UI dependencies for CI frontend-cache (#11313)	3 years ago
John Cowen	79b53ab23a	ui: Move the Role remove dialog to use InformedAction (#11298)	3 years ago
John Cowen	585f78f35a	ui: Fixup route blueprint to reflect project conventions (#11153)	3 years ago
Kenia	daec73e76c	ui: Topology - Fix up Default Allow and Permissive Intentions notices (#11216) ... * ui: Default allow notices test (#11240)	3 years ago
DillonStreator	03bc755162	ui: consistent empty state for intentions (#11261)	3 years ago
John Cowen	ed6918c6c2	ui: Improve dev-time SSO/OIDC visibility (#11248) ... This commit tries to make the development experience of working on our OIDC support a little more realistic, essentially by creating our own OIDC provider in our application (only during development builds). You can still provide a real OIDC provider to work with via our dev time environment/cookie variables as before, just now we default to the behaviour in this commit. Overall this makes it much easier to verify our OIDC support in the UI, and also opens up avenues for us to be able to test more scenarios that we couldn't before (for example not only successful logins, but also erroneous, potentially with multiple error reasons).	3 years ago
John Cowen	26c9d5c135	ui: Ignore erroneous ember/jQuery eslint warnings (#11238)	3 years ago
John Cowen	e5a15fdaac	ci: Don't focus install JS deps (#11267)	3 years ago
John Cowen	ebc8cbf957	ui: Fix up socket icon (#11234)	3 years ago
John Cowen	1cf008296d	docs: Fixes a typo in the UI readme docs (#11268)	3 years ago
Kenia	fca0e117e5	ui: Instructions for branch naming pattern (#11256) ... * Instructions for branch naming pattern * fix	3 years ago
John Cowen	baa377ddca	ui: Adds initial CRUD for partitions (#11188) ... * Add `is` and `test` helpers in a similar vein to `can` Adds 2 new helpers in a similar vein to ember-cans can: - `is` allows you to use vocab/phrases such as (is "something model") which calls isSomething() on the models ability. - `test` allows you to use vocab/phrases such as (test "is something model") or (test "can something model")which calls isSomething() / canSomething() on the models ability. Mostly using the is helper and the can helper. It's basically the is/can helper combined. * Adds TextInput component + related modifiers/helpers/machines/services (#11189) Adds a few new components/modifiers/helpers to aid building forms. - state-chart helper, used in lieu of a more generic approach for requiring our statecharts. - A few modifications to our existing disabled modifier. - A new 'validation' modifier, a super small form validation approach built to make use of state charts (optionally). Eventually we should be able to replace our current validation approach (ember-changeset-validations + extra deps) with this. - A new TextInput component, which is the first of our new components specifically to make it easy to build forms with validations. This is still a WIP, I left some comments in pointing out where this one would be progressed, but as we don't need the planned functionality yet, I left it where it was. All of this will be fleshed out more at a later date. Documentation is included for all of ^ * ui: Adds initial CRUD for partitions (#11190) Adds basic CRUD support for partitions. Engineering-wise probably the biggest takeaway here is that we needed to write very little javascript code to add this entire feature, and the little javascript we did need to write was very straightforwards. Everything is pretty much just HTML. Another note to make is that both ember-changeset and ember-data (model layer things) are now completely abstracted away from the view layer of the application. New components: - Consul::Partition::Form - Consul::Partition::List - Consul::Partition::Notifications - Consul::Partition::SearchBar - Consul::Partition::Selector See additional documentation here for more details New Route templates: - index.hbs partition listing/searching/filtering - edit.hbs partition editing and creation Additionally: There is some additional debug work here for better observability and to prevent any errors regarding our href-to usage when a dc is not available in our documentation site. Our softDelete functionality has been DRYed out a little to be used across two repos. isLinkable was removed from our ListCollection component for lists like upstream and service listing, and instead use our new is helper from within the ListCollection, meaning we've added a few more lighterweight templateOnly components. * ui: Exclude all debug-like files from the build (#11211) This PR adds **/*-debug.* to our test/prod excluded files (realised I needed to add test-support.js also so added that here as its more or less the same thing). Conditionally juggling ES6 static imports (specifically debug ones) for this was also getting a little hairy, so I moved it all to use the same approach as our conditional routes. All in all it brings the vendor build back down to ~430kb gzipped.	3 years ago
John Cowen	b8166de30d	ui: Replaces almost all remaining instances of SASS variables with CSS (#11200) ... From an engineers perspective, whenever specifying colors from now on we should use the form: ``` color: rgb(var(--tone-red-500)); ``` Please note: - Use rgb. This lets us do this like rgb(var(--tone-red-500) / 10%) so we can use a 10% opacity red-500 if we ever need to whilst still making use of our color tokens. - Use --tone-colorName-000 (so the prefix tone). Previously we could use a mix of --gray-500: $gray-500 (note the left hand CSS prop and right hand SASS var) for the things we need to theme currently. As we no longer use SASS we can't do --gray-500: --gray-500, so we now do --tone-gray-500: --gray-500. Just for clarity after that, whenever specifying a color anywhere, use rgb and --tone. There is only one reason where you might not use tone, and that is if you never want a color to be affected by a theme (for example a background shadow probably always should use --black) There are a 2 or 3 left for the code editor, plus our custom-query values	3 years ago
John Cowen	a9fe39e035	ui: Fix up blocking reconciliation for multiple models (#11237) ... > In the future, this should all be moved to each individual repository now, which will mean we can finally get rid of this service. This PR moves reconciliation to 'each individual repository'. I stopped short of getting rid of the service, but its so small now we pretty much don't need it. I'd rather wait until I look at the equivalent DataSink service and see if we can get rid of both equivalent services together (this also currently dependant on work soon to be merged) Reconciliation of models (basically doing the extra work to clean up the ember-data store and bring our frontend 'truth' into line with the actual backend truth) when blocking/long-polling on different views/filters of data is slightly more complicated due to figuring out what should be cleaned up and what should be left in the store. This is especially apparent for KVs. I built in a such a way to hopefully make sure it will all make sense for the future. I also checked that this all worked nicely with all our models, even KV which has never supported blocking queries. I left all that work in so that if we want to enable blocking queries/live updates for KV it now just involves deleting a couple of lines of code. There is a tonne of old stuff that we can clean up here now (our 'fake headers' that we pass around) and I've added that to my list of thing for a 'Big Cleanup PR' that will remove lots of code that we no longer require.	3 years ago
John Cowen	8b002d086a	ui: Address some Admin Partition FIXMEs (#11057) ... This commit addresses some left over admin partition FIXMEs 1. Adds Partition correctly to Service Instances 2. Converts non-important 'we can do this later' FIXMEs to TODOs 3. Removes some FIXMEs that I've double checked and addressed. Most of the remaining FIXMEs I'm waiting on responses to questions from the consul core folks for. I'll address those in a separate PR.	3 years ago
Kenia	5e7ef183ea	ui: Use of header default ACL policy (#11192) ... * Use of header default ACL policy * Update test for dc serializer	3 years ago
John Cowen	35a92e856b	ui: Make it hard to not URLEncode DataSource srcs/URIs (#11117) ... Our DataSource came in very iteratively, when we first started using it we specifically tried not to use it for things that would require portions of the @src="" attribute to be URL encoded (so things like service names couldn't be used, but dc etc would be fine). We then gradually added an easy way to url encode the @src="" attributes with a uri helper and began to use the DataSource component more and more. This meant that some DataSource usage continued to be used without our uri helper. Recently we hit #10901 which was a direct result of us not encoding @src values/URIs (I didn't realise this was one of the places that required URL encoding) and not going back over things to finish things off once we had implemented our uri helper, resulting in ~half of the codebase using it and ~half of it not. Now that almost all of the UI uses our DataSource component, this PR makes it even harder to not use the uri helper, by wrapping the string that it requires in a private URI class/object, that is then expected/asserted within the DataSource component/service. This means that as a result of this PR you cannot pass a plain string to the DataSource component without seeing an error in your JS console, which in turn means you have to use the uri helper, and it's very very hard to not URL encode any dynamic/user provided values, which otherwise could lead to bugs/errors similar to the one mentioned above. The error that you see when you don't use the uri helper is currently a 'soft' dev time only error, but like our other functionality that produces a soft error when you mistakenly pass an undefined value to a uri, at some point soon we will make these hard failing "do not do this" errors. Both of these 'soft error' DX features have been used this to great effect to implement our Admin Partition feature and these kind of things will minimize the amount of these types of bugs moving forwards in a preventative rather than curative manner. Hopefully these are the some of the kinds of things that get added to our codebase that prevent a multitude of problems and therefore are often never noticed/appreciated. Additionally here we moved the remaining non-uri using DataSources to use uri (that were now super easy to find), and also fixed up a place where I noticed (due to the soft errors) where we were sometimes passing undefined values to a uri call. The work here also led me to find another couple of non-important 'bugs' that I've PRed already separately, one of which is yet to be merged (#11105), hence the currently failing tests here. I'll rebase that once that PR is in and the tests here should then pass 🤞 Lastly, I didn't go the whole hog here to make DataSink also be this strict with its uri usage, there is a tiny bit more work on DataSink as a result of recently work, so I may (or may not) make DataSink equally as strict as part of that work in a separate PR.	3 years ago
John Cowen	7a5f335b7c	ui: Adds Flight icons to our icon set (#11097) ... * ui: Adds Flight icons to our icon set This commit doesn't actually add anything to the UI, rather just makes the flight icons available for use.	3 years ago
John Cowen	5e34d89f7f	ui: Hide create buttons for policies/roles/namespaces with readonly access (#10914) ... This PR adds a check to policy, role and namespace list pages to make sure the user has can write those things before offering to create them via a button. (The create page/form would then be a read-only form)	3 years ago
John Cowen	bce724baf1	ui: Don't show the CRD menu for read-only intentions (#11149) ... * ui: Don't show the CRD menu for read-only intentions The UI bug here manifests itself only when a user/token is configured to have read-only access to intentions. Instead of only letting folks click to see a read only page of the intention, we would show an additional message saying that the intention was read-only due to it being 'Managed by [a kubernetes] CRD'. Whilst the intention was still read only, this extra message was still confusing for users. This PR fixes up the conditional logic and further moves the logic to use ember-can - looking at the history of the files in question, this bug snuck itself in partly due to it being 'permission-y type stuff' previous to using ember-can and when something being editable or not was nothing to do with ACLs. Then we moved to start using ember-can without completely realising what IsEditable previously meant. So overall the code here is a tiny bit clearer/cleaner by adding a proper can view CRD intention instead of overloading the idea of 'editability'.	3 years ago
John Cowen	ce21c5109b	ui: Adds a set of basic unit tests for abilities (#11132)	3 years ago
John Cowen	581357c32a	ui: Remove info panel from the nspace menu when editing nspaces (#11130) ... * ui: Remove info panel form the nspace menu when editing nspaces	3 years ago
John Cowen	01c9dc5728	ui: Add initial partition support to intentions (#11129) ... * ui: Add initial partition support to intentions	3 years ago
John Cowen	eb00b45ef4	ui: Use last-child for selecting the desired nspace instead of nth-child (#11127)	3 years ago
John Cowen	e59b8ffc17	ui: Remove refresh-route action for session invalidation (#11105) ... * ui: Move action to the correct button for session invalidation * Remove refresh-route completely, its not needed	3 years ago
John Cowen	6bc8af364f	ui: Add initial i18n docs page (#10888)	3 years ago
John Cowen	61e69a9fe5	ui: Add partition parameter when clearing child-selector forms in ACLs (#11106)	3 years ago
John Cowen	ececa7da45	ui: Add an isDestroyed check for the MenuPanel component (#11104) ... This solves an occasionally flakey tests I see every so often	3 years ago
John Cowen	e088d8674c	ui: Remove legacy ACLs (#11096)	3 years ago
John Cowen	6e396e4456	ui: Gracefully recover from non-existent DC errors (#11077) ... * ui: Gracefully recover from non-existent DC errors This PR fixes what happens in the UI if you try to navigate to a non-existing DC. When we received a 500 error from an API response due to a non-existent DC, previously we would show a 404 error, which is what we were trying to convey. But in the spirit of the UI being a 'thin client', its probably best to just show the 500 error from the API response, which may help folks to debug any issues better. * Automatically set the CONSUL_DATACENTER_LOCAL env var for testing	3 years ago
John Cowen	cf638ee551	ui: Always show main navigation Key/Value link (#10916) ... * ui: Ignore response from API for KV permissions Currently there is no way for us to use our HTTP authorization API endpoint to tell us whether a user has access to any KVs (including the case where a user may not have access to the root KV store, but do have access to a sub item) This is a little weird still as in the above case the user would click on this link and still get a 403 for the root, and then have to manually type in the URL for the KV they do have access to. Despite this we think this change makes sense as at least something about KV is visible in the main navigation. Once we have the ability to know if any KVs are accessible, we can add this guard back in. We'd initially just removed the logic around the button, but then noticed there may be further related KV issues due to the nested nature of KVs so we finally decided on simply ignoring the responses from the HTTP API, essentially reverting the KV area back to being a thin client. This means when things are revisited in the backend we can undo this easily change in one place. * Move acceptance tests to use ACLs perms instead of KV ones	3 years ago
John Cowen	9c0233cf59	Revert "ui: Add partition parameter when clearing child-selector forms in ACLs" ... This reverts commit a670bde57d.	3 years ago
John Cowen	a670bde57d	ui: Add partition parameter when clearing child-selector forms in ACLs	3 years ago