171 Commits (a7c479e0739ba7a4599e5ccda81ef53cec3ee3ad)

Author	SHA1	Message	Date
John Cowen	14aa90e309	ui: Move repo services to repository/ folder and standardize naming (#4694) ... Repositories are a class of services to help with CRUD actions, most of the functionality is reused across various Models. This creates a new repository service that centralizes all this reused functionality. Inheritance via ember `Service.extend` is used as opposed to decorating via Mixins. 1. Move all repository services (and their tests) to a services/repository folder 2. Standardize on a singular name format 'node vs nodes' 3. Create a new 'repository' service to centralize functionality. This should be extended by 'repository' services	6 years ago
John Cowen	d98c1242dc	ui: Adds multi syntax linting to the code editor (#4814)	6 years ago
John Cowen	7d89e519a2	UI: New ACLs (#4789) ... UI to accompany the new ACLs APIs	6 years ago
John Cowen	297788408b	ui: Fix freetext searching within the nodes page (#4819) ... Essentially this was missing a call to `super`. The error unfortuantely didn't arise in the tests as it only errors when the node list has 4 items are more (the 4 columns), and the acceptence tests by change were only filling the page with 3 nodes for test purposes. I've bumped the amount of nodes up to 4 in the tests, which then causes the tests to fail, made the fix by adding the `super` call, and the tests now pass. I also tested the UI/text searching on a 10,000 node system, and everything now works as expected.	6 years ago
John Cowen	5f39bfd161	ui: Improve layout of node cards on large and small screens (#4761) ... 1. The grid based unhealthy cards are now clamped to only four wide maximum. This means that on larger screen the cards are much wider meaning you can view more information. Grid gutters are also clamped at a certain ideal width screen, remaining responsive for anything below this. 2. The healthy node columns are finally responsive following the same column rules as unhealthy nodes	6 years ago
John Cowen	461db096ed	ui: Add truncation to the `.Address` (#4760)	6 years ago
John Cowen	b75731aaec	ui: Cope with service names that contain slashes (#4756)	6 years ago
John Cowen	3410f5955a	ui: Reduce the breakpoint at which the smaller footer kicks in (#4723)	6 years ago
John Cowen	8ba1c549a9	ui: Move the text encoding polyfill to a a proper detecting polyfill (#4767)	6 years ago
John Cowen	9da8ad8f3d	UI: Package upgrades (#4740) ... Upgrade all patch and minor upgradeable packages, also uses `only` in ember-cli-build to reduce the included helpers from certain helper packages. Make some major version upgrades for some dev tools - husky - lint-staged - ember-cli-yadda - ember-cli-sass (also moved from node-sass to dart-sass) Minor tweak: spotted css file (instead of scss file), rename The move to `dart-sass`: dart-sass has been the primary implementation of sass for ~6 months and will receive updates earlier than libsass (ruby-sass itself is now deprecated) Other benefits include not having to recompile (via `npm rebuild` or similar) when switching platforms and an 'almost' javascript based solution. This update also alters some media queries that, whilst wouldn't compile anymore with either an updated libsass or dart-sass, where probably a little over complicated anyway, I've therefore made them similar to other breakpoints that made sense.	6 years ago
John Cowen	99c51c9f86	UI: Reduce in-development deprecations (also use toLocaleString) (#4677) ... Various ember addons produced deprecation messages, some in the browser console and some in terminal. Upgrading and replacing some of these has reduced this. Upgrades: - ember-collection - ember-computed-style Replacements: - ember-pluralize replaced with ember-inflector - ember-cli-format-number replaced with custom helper using standard `toLocaleString` Removing ember-cli-format-number also meant some further changes related to decimal places in the tomography graph, done using `toFixed` The ExternalSources background-images have also now been escaped correctly preventing in-development `console` warnings. The only deprecation warnings are now from ember-block-slots, only in terminal, making for a better development experience overall, especially now we have an empty browser console Also adds a `callIfType` 'helper util' which is a util specifically for helpers (it conforms to a helper argument signature) to be expanded upon later.	6 years ago
John Cowen	96508e556b	ui: Layout fix. Small padding additions to tables (#4701) ... 1. The 'Services' header need to be knocked ot the right slightly to line up properly with the service name when there are no external source icons. 2. Add a single space between ServiceName and ServiceID on the Node > [Services] tab table.	6 years ago
John Cowen	ece09e300d	UI: Tooltips and feedback-dialogs are the same thing - merge (#4678)	6 years ago
John Cowen	d0405ba8b9	UI: CSS Additions (mainly %frames) (#4623) ... * Move almost everything to use %frames * Fix pill styles of ACL types * Remove horizontal scrollbars from dom recycling scroller component * Make text areas look ok in Firefox * Remove ember-bulma-css * New form elements, break out %toggle * %button design tweaks * %form-element design tweaks * Better hashicorp logo * Small screen CSS improvements (#4624) 1. Reduce header size when there are no breadcrumbs 2. Make the filters toggleable, closed by default 3. Reduce the size of the footer on small screens 4. Hide all non-primary columns for forms 5. Slightly change the layout of various items, mainly buttons within forms 6. Make some confirmation dialogs work vertically on small screens. Guessing we might be better just using native confirmations on small screens	6 years ago
John Cowen	028875a732	UI: Set the CODE view as the default view for editing KV's (#4651) ... Sets the code toggle on the KV edit/create page to be on by default, we figured most people probably prefer this view. Also, previously we forced the KV toggle back to a default setting for every time you visited a KV form page. We've now changed this so that the KV code toggle button acts as a 'global' toggle. So whatever you set it as will be the same for every KV for the lifetime of your 'ember session' If we are to keep this, then consider saving this into localStorage settings or similar, added some thoughts in comments re: this as it's very likely to happen.	6 years ago
John Cowen	f54fd21946	ui: [BUGFIX] Intentions were showing the wrong notification on creation (#4658) ... The error notification was being shown on creation of an intention. This was as a result of #4572 and/or #4572 and has not been included in a release. This includes a fix, plus tests to try to prevent any further regression.	6 years ago
John Cowen	0757a08684	ui: Adds a `default` view helper for providing a default value (#4650) ... If the first value passed to the helper is an empty string or undefined then return the second value	6 years ago
John Cowen	5ea748005c	UI: External Source markers (#4640) ... 1. Addition of external source icons for services marked as such. 2. New %with-tooltip css component (wip) 3. New 'no healthcheck' icon as external sources might not have healthchecks, also minus icon on node cards in the service detail view 4. If a service doesn't have healthchecks, we use the [Services] tabs as the default instead of the [Health Checks] tab in the Service detail page. 5. `css-var` helper. The idea here is that it will eventually be replaced with pure css custom properties instead of having to use JS. It would be nice to be able to build the css variables into the JS at build time (you'd probably still want to specify in config which variables you wanted available in JS), but that's possible future work. Lastly there is probably a tiny bit more testing edits here than usual, I noticed that there was an area where the dynamic mocking wasn't happening, it was just using the mocks from consul-api-double, the mocks I was 'dynamically' setting happened to be the same as the ones in consul-api-double. I've fixed this here also but it wasn't effecting anything until actually made certain values dynamic.	6 years ago
John Cowen	981882d5ff	UI: Bugfix. Remove split view code editor (#4615) ... When adding an auto resizing (heightwise) code editor, the ivy-codemirror plugin seems to do this using more nested divs. This div had a horizontal scroller but couldn't be seen on some platforms (with hidden scrollbars). This commit makes the code editor slightly more usable and more visually correct by removing the scroll bar in this div to stop producing the 'split view look', yet keeping the horizontal scroller at the bottom of the code editor for when you enter code that is wider than the area. A max-width has also been added here to prevent the text area from growing off the side of the page. Another improvement to the code editor here is the addition of a nicer color for hightlighting text selection so its at least visible. Lastly, there was a way you could get the bottom horizontal scrollbar to overlay the code in the editor. This makes sure there is always some space at the bottom of the editor to make sure the code won't be obscured	6 years ago
John Cowen	e5f300dd21	UI: Bugfix. Move to a different TextEncoder/Decoder (#4613) ... 1. The previously used TextEncoder/Decoder (used as a polyfill for browsers that don't have a native version) didn't expose an encoder via CommonJS. Use a different polyfill that exposes both a decoder and an encoder. 2. The feature detection itself was flawed. This does a less error prone detection that ensures native encoding/decoding where available and polyfilled encoding/decoding where not available.	6 years ago
John Cowen	5e987965fa	UI: Bugfix. Fix code toggle in Safari (#4608) ... I'd mistakenly changed the checkbox event to listen to oninput, which works in Chrome and Firefox. Changed this back to onchange as it should be.	6 years ago
John Cowen	40e71f1b91	UI: Simplify/refactor the actions/notification layer (#4572) + (#4573) ... * Move notification texts to a slightly different layer (#4572) * Further Simplify/refactor the actions/notification layer (#4573) 1. Move the 'with-feedback' actions to a 'with-blocking-action' mixin which better describes what it does 2. Additional set of unit tests almost over the entire layer to prove things work/add confidence for further changes The multiple 'with-action' mixins used for every 'index/edit' combo are now reduced down to only contain the functionality related to their specific routes, i.e. where to redirect. The actual functionality to block and carry out the action and then notify are 'almost' split out so that their respective classes/objects do one thing and one thing 'well'. Mixins are chosen for the moment as the decoration approach used by mixins feels better than multiple levels of inheritence, but I would like to take this fuether in the future to a 'compositional' based approach. There is still possible further work to be done here, but I'm a lot happier now this is reduced down into separate parts.	6 years ago
John Cowen	b41cad6fdf	UI: CSS refactor (#4430) + Fullscreen Layout (#4435) ... * Begin refactoring CSS into component folders. Moved most components into layout/skin folders, left out a couple of ones I want to think about more. * Adjust grays based on recent Structure changes * Switch to fullscreen layout for lists and detail, left aligned forms (#4435) * Specifically use the 'actions_close' label, not just the :last-child (actions-group) * Replace some non-var-ed colours in vaults code skin, plus prefixing (black and white)	6 years ago
John Cowen	7bb35c4c78	UI: Repo layer integration tests (#4454) (#4563) ... ui: Repo layer integration tests for methods that touch the API Includes a `repo` test helper to make repetitive tasks easier, plus a injectable reporter for sending performance metrics to a centralized metrics system Also noticed somewhere in the ember models that I'd like to improve, but left for the moment to make sure I concentrate on one task at a time, more or less: The tests currently asserts against the existing JSON tree, which doesn't seem to be a very nice tree. The work at hand here is to refactor what is there, so test for the not nice tree to ensure we don't get any regression, and add a skipped test so I can come back here later	6 years ago
John Cowen	1b3d566a7a	UI: Begin unskipping some more trivial tests (#4574) ... WIP Unskip some lower level trivial tests. This is the beginning of work to unskip some of the more trivial tests that I'd skipped a while back (if the thing they are testing broke, they would have failed higher up in other acceptance tests). I'd rather keep the tests, as they do test things in a more isolated manner, and the plan was to always come back and work to unskip them time allowing. I didn't get to far into this work in progress here, but I'd rather merge what I've done all the same and come back at a later date and continue.	6 years ago
John Cowen	4ebd70e6cd	UI: Fixes healthy node listing resize on large portrait screens (#4564) ... 1. Split the resizing functionality of into a separate mixin to be shared across components 2. Add basic integration tests to prove that everything is getting called through out the lifetime of the app. I decided against unit testing as there isn't really any isolated logic to be tested, more checking that things are being called in the correct order etc i.e. the integration is correct. Adds assertion to with-resizing so its obvious to override `resize`	6 years ago
John Cowen	fde41467cf	ui: Adds bottom breathing space on the bottom of forms (#4433)	6 years ago
John Cowen	12811c0844	UI - Refactor Adapter.handleResponse (#4398) ... * Add some tests to check the correct GET API endpoints are called * Refactor adapters 1. Add integration tests for `urlFor...` and majority `handleResponse` methods 2. Refactor out `handleResponse` a little more into single/batch/boolean methods 3. Move setting of the `Datacenter` property into the `handleResponse` method, basically the same place that the uid is being set using the dc parsed form the URL 4. Add some Errors for if you don't pass ids to certain `urlFor` methods	6 years ago
John Cowen	0663bdbe90	Merge pull request #4410 from hashicorp/bugfix/gh-4175-service-ip ... UI - Bugfix: Show Service IP not the Node IP	6 years ago
John Cowen	e7a3235afc	Use `some` for the tags search instead of munging and searching	6 years ago
John Cowen	d510e1e6d3	Add basic searching by tags using the freetext search near-term	6 years ago
John Cowen	1d31bef0a4	Show the Service.IP address instead of the Node.IP address in detail	6 years ago
John Cowen	7202f6e771	Merge pull request #4387 from hashicorp/feature/ui-service-ids ... UI - Show/search by service ids	6 years ago
John Cowen	56d2052345	Merge pull request #4373 from hashicorp/bugfix/gh-4324-kv-trailing-slashes ... UI Bugfix - kv trailing slashes	6 years ago
John Cowen	310a038f87	Show Service.ID's throughout the app, allow searching by Service.ID ... 1. In the Services > Services detail page for both healthy and unhealthy nodes, also add searching by Service.ID here 2. In the Nodes > Node detail > [Services] tab only if its different from the Service name, add searching by Service.ID here	6 years ago
John Cowen	4179afae06	Instead of rewriting the API call, use ember to rewrite the URL ... We now essentially do 2 redirects if you hit a `folder/` 1. If you visit `/ui/dc1/kv/folder/`, `consul` will redirect you to `/ui/dc1/kv/folder` 2. Once redirected to `/ui/dc1/kv/folder` via a 301, use ember/history API to redirect you back to `/ui/dc1/kv/folder/`. Bit long winded, but achieves what we want without having to get stuck into `consul` itself to remove the 301 for the UI	6 years ago
John Cowen	887d4e9dde	If we are indexing or foldering then its going to be folder-like ... ...therefore add a trailing slash to the API call	6 years ago
John Cowen	fb91d05eb4	Don't assume the last item in the healthcheck listing is passing	7 years ago
John Cowen	25477761f7	Merge pull request #4343 from hashicorp/bugfix/gh-4336-update-empty-kv ... UI - Bugfix: Saving empty key/value pairs	7 years ago
John Cowen	c01fb37c46	Don't clone prev, there's no need	7 years ago
John Cowen	67402b3d26	Tests and comments regarding the previous 2 commits	7 years ago
John Cowen	b29546e578	Looking into atob functionality, consequence of Value: null ... The Consul API can pass through `Value: null` which does not get cast to a string by ember-data. This snowballs into problems with `atob` which then tried to decode `null`. There are 2 problems here. 1. `Value` should never be `null` - I've added a removeNull function to shallowly loop though props and remove properties that are `null`, for the moment this is only on single KV JSON responses - therefore `Value` will never be `null` which is the root of the problem 2. `atob` doesn't quite follow the `window.atob` API in that the `window.atob` API casts everything down to a string first, therefore it will try to decode `null` > `'null'` > `crazy unicode thing`. - I've commented in a fix for this, but whilst this shouldn't be causing anymore problems in our UI (now that `Value` is never `null`), I'll uncomment it in another future release. Tests are already written for it which more closely follow `window.atob` but skipped for now (next commit)	7 years ago
John Cowen	6a407a044e	Remove validation for presence of KV values	7 years ago
John Cowen	f85369c375	Add some navigation testing for back buttons and create buttons	7 years ago
John Cowen	649d777714	Acceptance test for using tokens from listing and detail pages	7 years ago
John Cowen	701d6a3a72	Move deleting to a top level feature	7 years ago
John Cowen	e0f7cdb128	Start purifying page objects	7 years ago
John Cowen	e3ce2a8beb	Lock Session invalidation acceptance test	7 years ago
John Cowen	ed286585e9	Add some low hanging intention tests, basically add intentions to others ... 1. There are various things tests that can just have intentions added into them, like filters and such like, add intentions to these 2. Start thinking about being able to negate steps easily, which will lead on to a cleanup of the steps	7 years ago
John Cowen	87a0ad9483	Merge pull request #4331 from hashicorp/feature/hedge-empty-node-ids ... Hedge for when consul sends nodes with an empty ID	7 years ago
John Cowen	b445df39bb	Use html5 oninput instead of onkeyup for native textual inputs ... This enables people to enter things using the mouse to paste for example, plus possible other things. As an aside it also answers my query regarding `fillIn` for testing, nothing needs to be actually _typed_ anymore! Doh	7 years ago
John Cowen	6398fb02c7	Ensure we catch empty ID's for single nodes also ... I don't think this would have a large effect on the UI whichever but best to make sure	7 years ago
John Cowen	a47a0b617f	Hedge for when consul sends nodes with an empty ID	7 years ago
John Cowen	8e7e224f3f	Encode all the hexcodes	7 years ago
mkeeler	6813a99081	Merge remote-tracking branch 'connect/f-connect'	7 years ago
John Cowen	5ac19b164e	Change the appearance of `*` for intention listings ... 1. Use 'All Services (*)' as opposed to '*' 2. Set 'Destination' in teh same bold font as 'Source' 3. Ensure you can search for all services by using '*' or 'All Services (*)'	7 years ago
John Cowen	9f8b87cdda	Check for NOT connect-proxy	7 years ago
John Cowen	cc3d578271	Filter Source and Destination menus by Kind	7 years ago
John Cowen	d61a09ea28	Fix linting typo, caused the selection of future services to break	7 years ago
John Cowen	f600d8fcc9	Move intentions menu item to after ACLs	7 years ago
John Cowen	b0ff68cd7d	Lint	7 years ago
John Cowen	e736b546f8	Icons in the filter bar	7 years ago
John Cowen	f4218b4bea	Validate all the things, all the time	7 years ago
John Cowen	ee3a32d771	More detailed error messages for duplicate intentions	7 years ago
John Cowen	ea3e4a720a	Only offer to create a future service if the name doesn't already exist	7 years ago
John Cowen	50507b34a5	Make sure future services are available when editing intentions	7 years ago
John Cowen	5575455053	Add support for specifying future services and wildcards	7 years ago
John Cowen	55bc45832e	Tie up real endpoints	7 years ago
John Cowen	24f6155efc	Switch to single selects rather than inputs	7 years ago
John Cowen	148621d98e	Have a look at power select for select elements	7 years ago
John Cowen	8023bb9b03	Add intentions actions mixin	7 years ago
John Cowen	9d6b9b7717	Remove SourceType for the moment	7 years ago
John Cowen	b38e5df630	WIP: First draft intentions ... 1. Listing, filtering by action and searching by source name and destination name 2. Edit/Create page, edits ping the API double fine, need to work through creates and deletes 3. Currently uses a `Precedence` intention keyname that doesn't yet exist in the real API	7 years ago
John Cowen	c3e92a236f	WIP: Logger	7 years ago
John Cowen	61e3fa3244	Make sure the dc menu is as useful as possible when things go wrong ... 1. Check the dc's model for both dcs list and the requested dc. 2. If the dc model doesn't exist use and empty array for dcs and a fake dc with the Name 'Error' as we can't even trust what is in the `paramsFor`	7 years ago
John Cowen	fd60f225ec	Merge branch 'master' into feature/ui-node-ports-for-service	7 years ago
John Cowen	b41d19c3c0	Merge pull request #4251 from hashicorp/bugfix/ui-null-tokens ... Ensure a blank/empty token is sent if the localStorage kv doesn't exist	7 years ago
John Cowen	30402b51b1	Merge pull request #4253 from hashicorp/feature/ui-clearer-tags ... UI - Clearer tags	7 years ago
John Cowen	3450aa81da	Merge pull request #4259 from hashicorp/feature/failing-ttl-test ... [BUGFIX] UI - Make sure lock session TTLs are shown	7 years ago
John Cowen	2e293f2697	Add a beforeModel hook at dc/index to auto transition to services	7 years ago
John Cowen	abf77b3c74	Merge remote-tracking branch 'shilov/master' into feature/failing-ttl-test	7 years ago
John Cowen	1881ec4824	Add a failing test to show that lock session TTL's are not shown	7 years ago
John Cowen	3f0560d098	Add missing scss file	7 years ago
John Cowen	0ad1d562db	Use the pill styling elsewhere... ... TODO: Probably change the %tag to %pill now I've remembered the word I was looking for!	7 years ago
John Cowen	0f6214d0ce	Make sure token is set to blank if nothing is typed in settings	7 years ago
John Cowen	d2d501a4a4	Add a reusable %tag and use it for the ACL 'tags' ready to use elsewhere	7 years ago
John Cowen	76017b69b5	Make sure the colon doesn't wrap with lots of tags	7 years ago
John Cowen	484f6ccaf2	Add port the service is running on for each node in the service view	7 years ago
John Cowen	8c5416ecaf	Ensure a blank token is sent if the localStorage kv doesn't exist	7 years ago
John Cowen	894667b87a	Merge pull request #4247 from hashicorp/feature/acl-sort-by-name ... UI - Sort ACL tokens by name	7 years ago
John Cowen	f960604bb5	Merge pull request #4242 from hashicorp/feature/ui-code-editor-autosizing ... Sets code editor height to be automatic (growable), with a reasonable min height	7 years ago
John Cowen	4d7f15ce34	Merge branch 'master' into bugfix/gh-4203-multi-tags	7 years ago
John Cowen	6949b22d5f	Sort ACL tokens by name	7 years ago
John Cowen	3c02e17fa7	Make sure the body background is white	7 years ago
Stanley Shilov	c06dfb3d2d	Sessions TTL should be of type string in ui-v2	7 years ago
John Cowen	5909d5695e	Sets code editor height to be auto, with a reasonable min height	7 years ago
John Cowen	04435dbdaf	Ensure all tags for a service are listed on the service detail page	7 years ago
John Cowen	bf4bcdd670	Find a way to test pre-ember load ... 1. Also add index.html things to test/index.html 2. Use content-for to hedge against keeping content in sync (requires an addon) 3. Test passes but only when run on its own, as we need to rely on content in the QUnit runner, theoretically it is not running our test in isolation. Skipping the test for the moment so we don't have a filaing test when all run together	7 years ago
John Cowen	5bee407319	Move healthcheck text down by 1px	7 years ago
John Cowen	11ec24fe48	Ensure startup/load image is removed if you hit an error on first load	7 years ago