Skip to content

feat: create structs representing Featurevisor instance and config manager #1

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

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

1e9y
Copy link

@1e9y 1e9y commented Nov 1, 2023

This is a work-in-progress branch for Instance and Config structs implementations, together with suggested public API for instantiating new Featuvisor client.

The Featurevisor instance is created with Config struct that responsible for reading the remote datafile config and holding methods for accessing underlying JSON field.

package config

// Config represents a parsed datafile config.
type Config interface {
Copy link
Member

@fahad19 fahad19 Nov 1, 2023

Choose a reason for hiding this comment

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

idea is to keep the interfaces/API as similar as the original JS SDK: https://github.com/featurevisor/featurevisor/tree/main/packages/sdk/src

this is how we are approaching Swift and Kotlin SDKs as well, helping keep it relatively easy to port bug fixes everywhere continuously.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants