"Version-5.5.0" release (#549)

* Fix non-existing avatar loading (#536)

* [CLEANUP] Auto build XP UI upon installing a package (#532)

* Remove lib/ files from the repo

* Remove redundant zappr config

* Limit the scope of installed package

* Remove useless on push build

* Add `data-testid` attribute to job related components (#539)

Why:
* This attribute is required for the Cypress tests in `xp-registration`

Co-authored-by: Marlo Vinall Richardson <[email protected]>

* [CLEANUP] Revert changes to keep xp-ui lib files in a repo

* Remove Contact UI and Email History code (#544)

* Debug code

* XP-3677 Remove Contact UI and Email History code

Co-authored-by: Nick Plekhanov <[email protected]>

* Add attribute hooks to job related components  (#540)

* Add missing `data-testid` attribute to Modal

Why:
* A test in `xp-registration` XP-3623 fails without this attribute

* Correct `data-testid` attributes

Why:
* Established naming convention dictates that camel case should be used

* Build the app

Co-authored-by: Marlo Vinall Richardson <[email protected]>
Co-authored-by: Nick Plekhanov <[email protected]>

Co-authored-by: Marlo <[email protected]>
Co-authored-by: Marlo Vinall Richardson <[email protected]>
Co-authored-by: Piotr Pasich <[email protected]>

Author: 4 people

.zappr.yaml

@@ -3,6 +3,9 @@
   "version": "2.0.0",
   "description": "UI component library for the XP platform.",
   "main": "lib/all.js",
+  "files": [
+    "lib"
+  ],
   "directories": {
     "doc": "docs"
   },

lib/.babelrc

@@ -115,7 +115,7 @@ class AccountConfirmation extends PureComponent<Props, void> {
         {(avatar || fullName) && (
           <span className={cx.avatar}>
             <Avatar
-              src={avatar || fullName}
+              src={avatar}
               alt={fullName}
               size={80}
             />

lib/.eslintrc

@@ -32,6 +32,15 @@ storiesOf('UI Components|AccountConfirmation', module)
   ))
 
 storiesOf('UI Components|AccountConfirmation/Debug', module)
+  .add('avatar src is not passed in', () => (
+    <AccountConfirmation
+      fullName='Monica Blacksmith'
+      email='[email protected]'
+      onContinue={action('onContinue')}
+      onSignOut={action('onSignOut')}
+    />
+  ))
+
   .add('missing props', () => (
     <AccountConfirmation />
   ))

lib/1672b3c098c1b00c2936bf3cec5257fd.png

@@ -189,7 +189,7 @@ class ApplicantScreenNotification extends PureComponent<Props, State> {
           exit: EXIT_TIMEOUT
         }}
       >
-        <div className={[cx.container, cx.color[this.props.type]].join(' ')}>
+        <div className={[cx.container, cx.color[this.props.type]].join(' ')} data-testid='xpui-applicantScreenNotification'>
           <div className={cx.content}>
             {this.renderIcon()}
             <div className={cx.text}>

lib/6f647ce328b13c96401708c6fcbdb237.gif

@@ -38,13 +38,13 @@ storiesOf('Core Components|Avatar/Use Cases', module)
   ))
 
 storiesOf('Core Components|Avatar/Debug', module)
-  .add('missing props (does component explode?)', () => (
-    <Avatar />
-  ))
   .add('with not reachable image source', () => (
     <Avatar
       src='https://x-team.com/some-path-that-doesn-exists.png'
       alt='X P'
       size={32}
     />
   ))
+  .add('missing props', () => (
+    <Avatar />
+  ))