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/decouple back button #211

Merged
merged 7 commits into from
Nov 8, 2023

Conversation

manansharma18
Copy link
Contributor

@manansharma18 manansharma18 commented Oct 29, 2023

Screenshot 2023-11-05 at 11 58 40 AM

resolves #169

Change Type (required)

Indicate the type of change your pull request is:
Adds a role property inside the metaData on the action asset. This property is used to decouple the back Icon from the back button.

  • patch
  • minor
  • major

@manansharma18 manansharma18 changed the title Feat/decouple back button feat/decouple back button Oct 29, 2023
@codecov
Copy link

codecov bot commented Oct 29, 2023

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (c3207f0) 82.18% compared to head (2049658) 82.19%.
Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #211      +/-   ##
==========================================
+ Coverage   82.18%   82.19%   +0.01%     
==========================================
  Files         132      132              
  Lines        4540     4544       +4     
  Branches     1117     1118       +1     
==========================================
+ Hits         3731     3735       +4     
  Misses        531      531              
  Partials      278      278              
Files Coverage Δ
...ference-assets/core/src/assets/action/transform.ts 89.47% <100.00%> (+2.80%) ⬆️
...eference-assets/react/src/assets/action/Action.tsx 14.28% <0.00%> (ø)

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

@@ -27,6 +27,9 @@ export interface ActionAsset<AnyTextAsset extends Asset = Asset>

/** Force transition to the next view without checking for validation */
skipValidation?: boolean;

/** boolean value to decide for the left anchor sign */
Copy link
Contributor

Choose a reason for hiding this comment

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

this docstring should be updated since it is a string now

@manansharma18 manansharma18 force-pushed the feat/decoupleBackButton branch from af35573 to af3a8b1 Compare November 8, 2023 17:10
@hborawski hborawski added the patch Increment the patch version when merged label Nov 8, 2023
@hborawski hborawski merged commit b3e436d into player-ui:main Nov 8, 2023
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch Increment the patch version when merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Core: Transform reference action asset role
3 participants