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

feat: add recommendations modal #189

Merged

Conversation

ahtesham-quraish
Copy link

@ahtesham-quraish ahtesham-quraish commented Aug 16, 2023

Description:
Add recommendations buttons and modal for painted door experiment

VAN-1601

It has been tested locally.

This is modal screenshot

image

Description:
Add recommendations buttons and modal for painted door experiment
VAN-1601
@zainab-amir zainab-amir changed the base branch from master to recommendations-painted-door August 16, 2023 07:59
@codecov
Copy link

codecov bot commented Aug 16, 2023

Codecov Report

Patch coverage: 95.45% and project coverage change: +0.03% 🎉

Comparison is base (ce269e8) 96.40% compared to head (743c233) 96.44%.

Additional details and impacted files
@@                       Coverage Diff                        @@
##           recommendations-painted-door     #189      +/-   ##
================================================================
+ Coverage                         96.40%   96.44%   +0.03%     
================================================================
  Files                               184      187       +3     
  Lines                              1725     1742      +17     
  Branches                            309      309              
================================================================
+ Hits                               1663     1680      +17     
  Misses                               57       57              
  Partials                              5        5              
Files Changed Coverage Δ
src/containers/LearnerDashboardHeader/messages.js 100.00% <ø> (ø)
src/widgets/RecommendationsPanel/messages.js 100.00% <ø> (ø)
...shboardHeader/CollapsedHeader/CollapseMenuBody.jsx 90.00% <66.66%> (+1.11%) ⬆️
src/components/ModalView/hooks.js 100.00% <100.00%> (ø)
src/components/ModalView/index.jsx 100.00% <100.00%> (ø)
src/components/ModalView/messages.js 100.00% <100.00%> (ø)
...s/LearnerDashboardHeader/CollapsedHeader/index.jsx 100.00% <100.00%> (ø)
...rs/LearnerDashboardHeader/ExpandedHeader/index.jsx 100.00% <100.00%> (ø)
src/widgets/RecommendationsPanel/LoadedView.jsx 100.00% <100.00%> (ø)
src/widgets/RecommendationsPanel/index.jsx 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ahtesham-quraish ahtesham-quraish force-pushed the ahtesham/VAN-1601/design branch 5 times, most recently from 904f819 to 08780d0 Compare August 16, 2023 11:01
Copy link
Contributor

@mubbsharanwar mubbsharanwar left a comment

Choose a reason for hiding this comment

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

There are some margin issues with the heading and buttons, we need to fix them.
Space between heading and text, and text and buttons should be according to Figma design.

Our design implementation
Screenshot 2023-08-16 at 2 48 27 PM

src/components/ModalView/index.jsx Show resolved Hide resolved
src/components/ModalView/index.jsx Outdated Show resolved Hide resolved
src/containers/LearnerDashboardHeader/hooks.js Outdated Show resolved Hide resolved
src/containers/LearnerDashboardHeader/hooks.js Outdated Show resolved Hide resolved
src/widgets/RecommendationsPanel/hooks.js Outdated Show resolved Hide resolved
@ahtesham-quraish ahtesham-quraish force-pushed the ahtesham/VAN-1601/design branch 2 times, most recently from 005d10e to 4ee6543 Compare August 16, 2023 12:09
Copy link
Contributor

@mubbsharanwar mubbsharanwar left a comment

Choose a reason for hiding this comment

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

Looks good.

@ahtesham-quraish ahtesham-quraish force-pushed the ahtesham/VAN-1601/design branch 3 times, most recently from 0ad5467 to fe88e69 Compare August 16, 2023 16:05
@ahtesham-quraish ahtesham-quraish merged commit 3bad489 into recommendations-painted-door Aug 16, 2023
4 of 5 checks passed
@ahtesham-quraish ahtesham-quraish deleted the ahtesham/VAN-1601/design branch August 16, 2023 16:23
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.

2 participants