Skip to content

EM-1658: Buy Box Widget #517

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

Merged
merged 5 commits into from
Jul 26, 2017
Merged

Conversation

MKwenhua
Copy link
Contributor

@MKwenhua MKwenhua commented Jul 14, 2017

Purpose:

To create a new Widget data type buy_box_widget so that POs can easily edit and change Buy Boxes to a greater degree and side step sprint tasks for rolling out test pages.

JIRA:

https://cb-content-enablement.atlassian.net/browse/EM-1658

Steps to Take On Prod

Before changes to employer hit prod
Copy the example Yaml files in Employer /config/data and paste into the Buy Box widget in cortex for these pages:

  • resume-database is file search_rdbs_buy_box_example.yml
  • resume-database-access is file search_access.yml
  • resume-database-search is file search_rdbs_search.yml
  • post-jobs is file jobs_buy_box_example.yml

Changes:

  • Changes to setup

    • migrate databases
  • Architectural changes

    • n/a
  • Migrations

    • yes run migration for new data type
  • Library changes

    • n/a
  • Side effects

    • n/a

Screenshots

  • Before

  • After

QA Links:

Cortex: http://web.cortex-4.development.c66.me/
Employer Dev Servers:

How to Verify These Changes

Relevant PRs/Dependencies:

cortex-cms/cortex-snippets-client-ruby#32

Additional Information

You may notice I added some tabs for the widgets which was not in the AC but I feel is necessary for ease of use as the number of widgets increase.

@MKwenhua MKwenhua merged commit 246d1f4 into legacy-develop Jul 26, 2017
@MKwenhua MKwenhua deleted the topic/EM-1658/Buy-Box-Widget branch July 26, 2017 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants