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.

Sign up for GitHub

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

Jump to bottom

Added search bar in policy files #5307

Merged
merged 5 commits into from
Jul 5, 2021
Merged

Added search bar in policy files #5307

merged 5 commits into from
Jul 5, 2021

Conversation

chaitali-mane
Copy link
Contributor

@chaitali-mane chaitali-mane commented Jun 30, 2021 •
edited
Loading

Signed-off-by: Chaitali Mane cmane@progress.com

🔩 Description: What code changed, and why?

We need search functionality in policy files list

⛓️ Related Resources

#5240

👍 Definition of Done

Added search bar functionality for policy files

👟 How to Build and Test the Change

STEP 1
inside the hab studio

[default:/src:0]# build components/automate-ui-devproxy/
[default:/src:0]# start_automate_ui_background
[default:/src:0]# start_all_services

STEP 2
open new window
go to automate UI path

$ cd components/automate-ui
and run the command 

npm run serve:hab

Steps for Policyfiles tab:
Go To Infrastructure -> Chef Servers -> click server name-> click on org name -> Policyfiles.

For cypress Build
https://github.com/chef/automate/tree/master/e2e

✅ Checklist

All PRs must tick these:

With occasional exceptions, all PRs from Progress employees must tick these:

  • Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
  • Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
  • Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
  • Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
  • Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

  • Tests added/updated? (all new code needs new tests)
  • Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.

📷 Screenshots, if applicable

  1. Search bar
    search-policy

  2. No data after search
    search-policy1

@netlify
Copy link

netlify bot commented Jun 30, 2021 •
edited
Loading

👷 Deploy Preview for chef-automate processing.

🔨 Explore the source changes: 20f2082

🔍 Inspect the deploy log: https://app.netlify.com/sites/chef-automate/deploys/60ddd35b72547100082e95ab

@chaitali-mane chaitali-mane self-assigned this Jun 30, 2021
@chaitali-mane
added search bar in policy files
4f95ee0 
Signed-off-by: chaitali-mane <cmane@progress.com>
@chaitali-mane
Added cypress test cases
7858ed6 
Signed-off-by: chaitali-mane <cmane@progress.com>
@chaitali-mane
minor change
d9000b0 
Signed-off-by: chaitali-mane <cmane@progress.com>
@chaitali-mane
Added code for pagination updation
3829f6e 
Signed-off-by: chaitali-mane <cmane@progress.com>
@chaitali-mane
Added unit tests
20f2082 
Signed-off-by: chaitali-mane <cmane@progress.com>
@sonarcloud
Copy link

sonarcloud bot commented Jul 1, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@kalroy kalroy merged commit 6350919 into master Jul 5, 2021
@kalroy kalroy deleted the Chaitali/search_policy branch July 5, 2021 13:15
@dishanktiwari2501 dishanktiwari2501 mentioned this pull request Jul 7, 2021
Merged
9 tasks
kalroy added a commit that referenced this pull request Jul 15, 2021
@dishanktiwari2501 @chaitali-mane
@tduffield @vivekshankar1 @PratyakshaSinha @kalroy @chef-expeditor @chef-ci @iamazzeez @himanshi-chhabra @Venkatesh-rengasamy
Update Screenshots, add content to Cookbook section & add Policyfile …
7ed9771 
…section to Chef Infra Server (#5316)

* created a list of headers to be added to the cookbook section in chef infra

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* added the cookbook section to the chef infra server page

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* minor fixes on cookbook section

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* images updated

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* Added search bar in policy files (#5307)

* added search bar in policy files

Signed-off-by: chaitali-mane <cmane@progress.com>

* Added cypress test cases

Signed-off-by: chaitali-mane <cmane@progress.com>

* minor change

Signed-off-by: chaitali-mane <cmane@progress.com>

* Added code for pagination updation

Signed-off-by: chaitali-mane <cmane@progress.com>

* Added unit tests

Signed-off-by: chaitali-mane <cmane@progress.com>

* Ensure /usr/local/bin is in PATH (#5311)

* Ensure /usr/local/bin is in PATH

Some platforms will not have /usr/local/bin in their PATH

Signed-off-by: Tom Duffield <github@tomduffield.com>

* Have install script create chef-ci user

Signed-off-by: Tom Duffield <github@tomduffield.com>

* added missing brackets to doc (#5318)

Signed-off-by: Vivek Shankar <vshankar@progress.com>

* added policy file section to the chef infra server page

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* Event Service document (#5214)

* Event feed document

Signed-off-by: PratyakshaSinha <email_address@domain.com>

* Event Feed document formatting

Signed-off-by: Kallol Roy <karoy@progress.com>

* Adding config possibilities

Signed-off-by: Kallol Roy <karoy@progress.com>

Co-authored-by: PratyakshaSinha <email_address@domain.com>
Co-authored-by: Kallol Roy <karoy@progress.com>

* edited images for the chef infra page

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* Bump automate-compliance-profiles (#5297)

This pull request was triggered automatically via Expeditor.

This change falls under the obvious fix policy so no Developer Certificate of Origin (DCO) sign-off is required.

Co-authored-by: Chef Expeditor <chef-ci@chef.io>

* add some more verbage to describe requirements for deep filtering (#5266)

* add some more verbage to discribe requirements for deep filtering

Signed-off-by: Rick Marry <rick.marry@gmail.com>

* Docs edits

Signed-off-by: kagarmoe <kgarmoe@chef.io>

* even shorter

Signed-off-by: kagarmoe <kgarmoe@chef.io>

* shortest

Signed-off-by: kagarmoe <kgarmoe@chef.io>

* word choice

Signed-off-by: kagarmoe <kgarmoe@chef.io>

* symmetry

Signed-off-by: kagarmoe <kgarmoe@chef.io>

Co-authored-by: kagarmoe <kgarmoe@chef.io>

* ldap server setup dev-docs update (#5203)

* ldap server setup dev-docs update

* Changed some formats and rewrote some comments

Signed-off-by: Kallol Roy <karoy@progress.com>

Co-authored-by: Kallol Roy <karoy@progress.com>

* minor fixes

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* minor fixes in cookbook section

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* minor fixes in policyfile section

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* Update components/docs-chef-io/content/automate/chef_infra_server.md

Co-authored-by: Kimberly Garmoe <kgarmoe@chef.io>

* added image for the policyfile tab

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* update headers and pointers in cookbook section

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* Added search text changes. (#5326)

Signed-off-by: Himanshi Chhabra <hchhabra@progress.com>

* fixes in chef infra section

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* updated image of policyfiles section

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* Removed bintray dependencies from deployment (#5328)

* Removed bintray dependecies from deployment

Signed-off-by: Kallol Roy <karoy@progress.com>

* New bundle creator

Signed-off-by: Kallol Roy <karoy@progress.com>

* Removed bin from test

Signed-off-by: Kallol Roy <karoy@progress.com>

* Removed date versioned hab from test

Signed-off-by: Kallol Roy <karoy@progress.com>

* Getting Binlink back

Signed-off-by: Kallol Roy <karoy@progress.com>

* Migrate away from Bintray for migration

Signed-off-by: Kallol Roy <karoy@progress.com>

* Added delete option in policy files (#5321)

* Added UI for delete part

Signed-off-by: chaitali-mane <cmane@progress.com>

* Added API integration

Signed-off-by: chaitali-mane <cmane@progress.com>

* Added test cases

Signed-off-by: chaitali-mane <cmane@progress.com>

* Updated API integration code

Signed-off-by: chaitali-mane <cmane@progress.com>

* minor change

Signed-off-by: chaitali-mane <cmane@progress.com>

* Minor change

Signed-off-by: chaitali-mane <cmane@progress.com>

* Minor change

Signed-off-by: chaitali-mane <cmane@progress.com>

* Security Improvements in Load balancer (#5286)

* Enable HSTS policy for the routes which does not have the header set as of now.

Signed-off-by: Kallol Roy <karoy@progress.com>

* More security fixes

Signed-off-by: Kallol Roy <karoy@progress.com>

* Review changes are done

Signed-off-by: Venkatesh-rengasamy <vrengasa@progress.com>

Co-authored-by: Venkatesh-rengasamy <vrengasa@progress.com>

* Venkatesh/telemetry preference save UI (#5284)

* Telemetry Pref is restored while login

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleaning - remove unnecessary changes

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleaning - remove unnecessary changes

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Telemetry Preference issue

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleaning

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Show telemetry warning message if preference is disabled

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleanup

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Restrict unnecessary api calls for segment write key

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* add static warning message with checkbox

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* update alignment

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* update alignment

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* update alignment

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Update warning message and its alignment

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleanup

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* anchor tag security issue in pipeline

Signed-off-by: Venkatesh-rengasamy <vrengasa@progress.com>

* Telemetry Pref is restored while login

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleaning - remove unnecessary changes

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleaning - remove unnecessary changes

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Telemetry Preference issue

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleaning

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Show telemetry warning message if preference is disabled

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleanup

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Restrict unnecessary api calls for segment write key

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* add static warning message with checkbox

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* update alignment

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* update alignment

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* update alignment

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Update warning message and its alignment

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* Code cleanup

Signed-off-by: Venkatesh-rengasamy <Venkatesh.Rengasamy@progress.com>

* anchor tag security issue in pipeline

Signed-off-by: Venkatesh-rengasamy <vrengasa@progress.com>

* upgrade highlight.js version

Signed-off-by: Venkatesh-rengasamy <vrengasa@progress.com>

* upgrade typedoc version for build error

Signed-off-by: Venkatesh-rengasamy <vrengasa@progress.com>

* fix protactor failed case - telemetry

Signed-off-by: Venkatesh-rengasamy <vrengasa@progress.com>

* revert typedoc version changes (#5354)

Signed-off-by: Venkatesh-rengasamy <vrengasa@progress.com>

* minor fixes for content tab

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* created a list of headers to be added to the cookbook section in chef infra

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* added the cookbook section to the chef infra server page

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* minor fixes on cookbook section

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* images updated

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* added policy file section to the chef infra server page

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* edited images for the chef infra page

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* minor fixes

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* minor fixes in cookbook section

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* minor fixes in policyfile section

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* fixing conflicts

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* added image for the policyfile tab

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* update headers and pointers in cookbook section

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* fixing conflicts

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* updated image of policyfiles section

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

* minor fixes for content tab

Signed-off-by: Dishank Tiwari <dtiwari@progress.com>

Co-authored-by: Chaitali Mane <71449322+chaitali-mane@users.noreply.github.com>
Co-authored-by: Tom Duffield <tom@chef.io>
Co-authored-by: vivekshankar1 <59958706+vivekshankar1@users.noreply.github.com>
Co-authored-by: Pratyaksha Sinha <65227203+PratyakshaSinha@users.noreply.github.com>
Co-authored-by: PratyakshaSinha <email_address@domain.com>
Co-authored-by: Kallol Roy <karoy@progress.com>
Co-authored-by: chef-expeditor[bot] <49165653+chef-expeditor[bot]@users.noreply.github.com>
Co-authored-by: Chef Expeditor <chef-ci@chef.io>
Co-authored-by: Rick Marry <rmarry@chef.io>
Co-authored-by: kagarmoe <kgarmoe@chef.io>
Co-authored-by: Abdul Azeez <aazeez@progress.com>
Co-authored-by: Himanshi Chhabra <61003053+himanshi-chhabra@users.noreply.github.com>
Co-authored-by: Venkatesh-rengasamy <vrengasa@progress.com>
Co-authored-by: Venkatesh-rengasamy <84014872+Venkatesh-rengasamy@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@himanshi-chhabra himanshi-chhabra himanshi-chhabra approved these changes

@Venkatesh-rengasamy Venkatesh-rengasamy Venkatesh-rengasamy approved these changes

Assignees

@chaitali-mane chaitali-mane

Labels
acceptance: verified automate-infra-proxy automate-ui ui
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@chaitali-mane @himanshi-chhabra @Venkatesh-rengasamy @kalroy