1Password / op-js

A JS library powered by the 1Password CLI
https://developer.1password.com/docs/cli
MIT License
94 stars 8 forks source link

Fixed #105 - Section interface should have a label field #141

Closed anujchoudhary-17 closed 1 year ago

anujchoudhary-17 commented 1 year ago

Sorry for the chaos in the last PR. Added label string as an optional variable field in the Section interface. In the op-js/src/index.ts file I have made the following changes. Please review it and let me know if any further changes are required :) image

@jodyheavener Are there any other issues you feel like I can work one? Please let me know. Would love to work on new issues

Thank you

jodyheavener commented 1 year ago

Hi @anujchoudhary-17 this looks great, thank you for fixing this!

In terms of other issues, it would be best to review issues in our various open source repos and contribute where you feel like you might do best. Some repos include: