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

Run eligibility check feature (#2463) #2477

Merged
merged 2 commits into from
Nov 17, 2023
Merged

Run eligibility check feature (#2463) #2477

merged 2 commits into from
Nov 17, 2023

Conversation

sri49
Copy link

@sri49 sri49 commented Nov 8, 2023

  • Eligibility check backend changes (Eligibility check backend changes #2460)

  • adds run eligibility check controller action and routes

  • adds specs

  • adds run eligibility route

  • Eligibility check authorization (Eligibility check authorization #2465)

  • adds pundit policy for admin access only

  • fixes typo

  • fixes spec failures

  • Run eligibility check front (Run eligibility check front #2464)

  • Implement eligibility checks in front-end UI

  • removed placeholder button

  • Fixed failing specs

  • fixed mistyped javascript

  • Rails renders button_tag as a form with an input field, why selector was not being found. Working specs

  • Added Specs for Eligibility Check Partial

  • Moved Eligibility Spec to its correct directory

  • Added Translations for Run Eligibility button

  • resolved rubocop errors for files that aren't slim templates

  • fixed more rubocop issues, more are failing

  • removed comment

  • fixed route mispell

  • Added translations for the response text

  • fixed UI breaking from translations update


  • adds condition to display the button during OE period

  • PR revisions

  • fixes the row alignment issue


Master Redmine ticket

Child Redmine ticket(s)

  • ()
  • ()

Local build result

bundle exec rake parallel:spec[4]
bundle exec cucumber

Latest rebase/merge tag


Data ticket(s) Followup

  • (redmine links here - optional)

Related Pull Requests

  • (github links here - optional)

TODOs / Notes

Peer Review

  • (For code review)

Functional Testing

  • (For testing locally)

Deployment

  • (for release manager and/or build manager)

* Eligibility check backend changes (#2460)

* adds run eligibility check controller action and routes

* adds specs

* adds run eligibility route

* Eligibility check authorization (#2465)

* adds pundit policy for admin access only

* fixes typo

* fixes spec failures

* Run eligibility check front (#2464)

* Implement eligibility checks in front-end UI

* removed placeholder button

* Fixed failing specs

* fixed mistyped javascript

* Rails renders button_tag as a form with an input field, why selector was not being found.  Working specs

* Added Specs for Eligibility Check Partial

* Moved Eligibility Spec to its correct directory

* Added Translations for Run Eligibility button

* resolved rubocop errors for files that aren't slim templates

* fixed more rubocop issues, more are failing

* removed comment

* fixed route mispell

* Added translations for the response text

* fixed UI breaking from translations update

---------

Co-authored-by: Utkarsh Shukla <utkarsh7989@gmail.com>

* adds condition to display the button during OE period

* PR revisions

* fixes the row alignment issue

---------

Co-authored-by: TonyHasIdeas <118292503+TonyHasIdeas@users.noreply.github.com>
Co-authored-by: Antonio Irizarry <antonio.irizarry@ideacrew.com>
Co-authored-by: Sri Harsha <sriharsha.poosa@gmail.com>
* adds wellsense flash message in benefit package and plan shopping pages

* adds translation

* adds wellsense flash message in bqt
Copy link

@vkghub vkghub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sri49 sri49 merged commit 7a74ce3 into master Nov 17, 2023
18 checks passed
@sri49 sri49 deleted the rc-release-5.7.73 branch November 17, 2023 03:45
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.

4 participants