Page Comparison

5 minutes

Ensure branches are up to date before merge?

Brian Smith

• Merging this led to failing tests that would have been caught if the branch had been rebased first

5 mins

Frontend-app-learner-dashboard Banner component

Mena Hassan Cindy Nguyen

In frontend-app-learner-dashboard it looks like Alert is being used to do what is essentially the function of Card status. See the ‘Grade required to pass the course: 60%’ banner under Demonstration Course here:

• Is there something about this usage of Alert that is being used because Card status isn’t satisfying another need here?

  • If so, how can we/how should we expand the functionality of Card status to cover use cases like this

5 min

Frontend-app-learner-dashboard EmailLink component

Cindy Nguyen Mena Hassan

Currently, EmailLink is covering for the case of if there isn’t an address return NULL

Should we make MailtoLink in Paragon already handle the use case of covering if there isn't an email address provided? 

This is how it’s being used within the MFE. Is this a pattern that’s ok? If so, how can we better support that pattern directly within Paragon? If not, what should we do instead?

• Certificatebanner.jsx line 29:

  Code Block
  const emailLink = address => address && <MailtoLink to={address}>{address}</MailtoLink>;

10 mins

Demonstrate and get accessibility feedback on Marketplace

Kaleb Davenport

The navigation can be found at https://www.edx.org/?mega=true , and we would love to get any and all feedback about how we can make the interaction patterns more comfortable and easy to browse using accessibility tools.

10-20 minutes

Design tokens: what’s left

Adam Stankiewicz

https://openedx.atlassian.net/wiki/spaces/BPL/pages/3898572801/Design+tokens+What+s+left