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.

Sign up for GitHub

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

Jump to bottom

🍱 Add "Open Lock" Icon (#26) #83

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

🍱 Add "Open Lock" Icon (#26) #83

wants to merge 1 commit into from
+32 −0

Conversation

mistryrn
Copy link
Contributor

Description of changes

Type of Change

  • Bug
  • Styling
  • New Feature

Checklist before requesting review:

  • Design (select one):
    • Matches design:
      • component sizes, spacing, and styles
      • font size, weight, colour
      • spelling has been double checked
    • No design provided, screenshot of changes included in PR
    • No changes to UI
  • UI Kit (select one):
    • New Components with storybook stories created
    • Modified Existing components and updates stories
    • No new UIKit components or changes
  • Feature is minimally responsive
  • Manual testing of UI feature
  • Add copyrights to new files
  • Connected ticket to PR

justincorrigible
justincorrigible approved these changes Oct 20, 2022
View reviewed changes
Copy link
Member

@justincorrigible justincorrigible left a comment

Choose a reason for hiding this comment

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

much lock. such safe 🔒

@joneubank
Copy link
Member

Any reason why this is approved and not merged? @mistryrn should this be merged in?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@justincorrigible justincorrigible justincorrigible approved these changes

@ciaranschutte ciaranschutte Awaiting requested review from ciaranschutte

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mistryrn @joneubank @justincorrigible