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

Sync providers: extract into its own folder #3000

Open
9 tasks
Tracked by #2938
cuserox opened this issue Jul 16, 2024 · 1 comment · May be fixed by #3092
Open
9 tasks
Tracked by #2938

Sync providers: extract into its own folder #3000

cuserox opened this issue Jul 16, 2024 · 1 comment · May be fixed by #3092

Comments

@cuserox
Copy link
Contributor

cuserox commented Jul 16, 2024

Context 📝

The implementation of sync providers in the plugin is repetitive and can be optimized. The aim of this task is to gather together as much of the UI & logic into its own realm, to set a base that allows us to publish as an independent package that can be flexible enough for future providers.

Acceptance Criteria 🛠️

  • Move all related constants, methods and component into a single folder in the plugin
  • Improve tests

🎯 A package that lets users create/choose a sync provider, and then have a button that Import variables which would basically just read local variables from Figma, and push those to the repository. So in effect we'd have all of these parts in the Figma plugin already, but we want those to be part of a new plugin that does just that. As in it'd be one-directional for now (pushing to git), but eventually would do both.

  1. cuserox
@cuserox cuserox self-assigned this Jul 16, 2024
@six7 six7 added this to the Improved Developer Experience milestone Jul 22, 2024
@cuserox cuserox changed the title Sync providers: create a shared handler to easily switch methods Sync providers: extract into its own folder Jul 22, 2024
@cuserox
Copy link
Contributor Author

cuserox commented Aug 20, 2024

@cuserox cuserox linked a pull request Aug 29, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🕜 In Progress
Development

Successfully merging a pull request may close this issue.

3 participants