Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge in #38

Merged
merged 128 commits into from
Feb 7, 2018
Merged

Merge in #38

merged 128 commits into from
Feb 7, 2018

Conversation

dkirillov
Copy link

erikmunson and others added 30 commits January 16, 2018 08:23
adds support for preconfigured webhook parameters. these parameters can
be set in orca's config and are then exposed to users. instead of
expecting users to fillout an entire payload, these parameters can be
used to expose the important parts and abstract the other configuration
away.
- feat(amazon/serverGroup): warn that scaling policies will not work when capacity is pinned
…ing fa-cog icons with new spinner (spinnaker#4630)

* Replaced cogs with new spinner

* removing space around 'finding' phrases + updating some of the spinners to be larger

* removing unnecessary spaces
Simplify interfaces and post-search logic.
Simplify Search v2 service logic.
Remove IFilterType 'modifier' in favor of 'key'; rename 'text' to 'name'
robfletcher and others added 28 commits February 1, 2018 17:46
…ing (spinnaker#4775)

- Refactor ISearchResultType to SearchResultType class

- Remove PostSearchResultSearcherRegistry (consolidate with SearchResultType)
- Remove SearchResultHydratorRegistry (consolidate with SearchResultType)
- Remove ExternalSearchRegistry (consolidate with SearchResultType)

- Migrate ClusterPostSearchResultSearcher and ClusterSearchResultHydrator to a ClustersSearchResultType
- Remove ServerGroupSearchResultHydrator
- Remove ApplicationPostSearchResultSearcher in favor of fetching /search?type=applications
- Migrate individual SearchResultType(s) to classes (i.e., ApplicationSearchResultType)

* Updates based on PR comments, and Add missing SearchResultType files
@dkirillov dkirillov merged commit b27eec2 into master Feb 7, 2018
@dkirillov dkirillov deleted the merge-in branch February 7, 2018 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.