Skip to content
This repository was archived by the owner on Sep 28, 2023. It is now read-only.

feat: Update readme with Feature Key Tracking feature #13

Merged
merged 7 commits into from
Oct 7, 2020

Conversation

yavorona
Copy link
Contributor

This pr updates README.md with feature key tracking feature

README.md Outdated

This extension is compatiable with 1.34 and higher of vscode.
This extension also evaluates the use of Optimizely's public APIs (e.g. getFeatureVariableJSON(feature_key, userId, attributes)) in active .js and .ts files, and highlights Feature Keys that do not exist in datafile.
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd reword as:
This extension also evaluates if parameter values are valid for Optimizely's public SDK methods (for example, getFeatureVariable(feature_key, userID, attributes), or isFeatureEnabled(feature_key)) for Javascript and Typescript. If a given parameter is invalid (i.e., doesn't exist in the datafile), this extension highlights it and displays a warning on hover.

Copy link
Contributor Author

@yavorona yavorona Oct 6, 2020

Choose a reason for hiding this comment

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

I think this is great! I would just specify what parameter values being evaluated, which are only feature keys at the moment:

This extension also evaluates if provided feature keys are valid for Optimizely's public SDK methods (for example, getFeatureVariable(feature_key, userID, attributes), or isFeatureEnabled(feature_key, userID)) for Javascript and TypeScript. If a given feature_key is invalid (i.e., doesn't exist in the datafile), the extension highlights it and displays a warning on hover.

README.md Outdated
@@ -1,8 +1,10 @@
# Optimizely VSCode Extension

The Optimizely VSCode extension lets you access Feature Keys, Experiment Keys and Feature Variables via auto-complete for javascript typescript and react. If not using these languages, you can use the keyboard shortcuts below to insert your experiment and feature keys as well as attribute and event keys.
The Optimizely VSCode extension lets you access Feature Keys, Experiment Keys and Feature Variables via auto-complete for javascript, typescript and react. If not using these languages, you can use the keyboard shortcuts below to insert your experiment and feature keys as well as attribute and event keys.
Copy link
Contributor

Choose a reason for hiding this comment

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

The Optimizely VSCode extension autocompletes:

  • feature keys
  • experiment keys
  • feature variables
    in Javascript, Typescript, and React.

QUESTION: it's not clear if In .js, .ts, and react, you can also autocomplete attribute and event keys without keyboard shortcuts? if you can, please add them to the list and modify the second sentence:
In other languages, you can insert the preceding list of elements using the following keyboard shortcuts.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I believe we can, so they should be added to the list. @thomaszurkan-optimizely WDYT?

Copy link
Contributor

Choose a reason for hiding this comment

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

events and attributes don't auto-complete.

Copy link
Contributor Author

@yavorona yavorona Oct 7, 2020

Choose a reason for hiding this comment

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

I added back events keys since we added it in our last #14 pr

@yavorona yavorona requested a review from fscelliott October 7, 2020 00:09
Copy link
Contributor

@thomaszurkan-optimizely thomaszurkan-optimizely left a comment

Choose a reason for hiding this comment

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

nit

README.md Outdated
This extension is compatiable with 1.34 and higher of vscode.
In other languages, you can insert the preceding list of elements using the following keyboard shortcuts.

This extension also evaluates if provided feature keys are valid for Optimizely's public SDK methods (for example, getFeatureVariable(feature_key, userID, attributes), or isFeatureEnabled(feature_key, userID)) for Javascript and TypeScript. If a given feature_key is invalid (i.e., doesn't exist in the datafile), the extension highlights it and displays a warning on hover.
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we change this slightly to say something like: "(i.e. doesn't exist in the datafile for the sdk key used to configure)."

@yavorona yavorona force-pushed the pnguen/update-readme branch from bf8038e to 69f2180 Compare October 7, 2020 18:35
Copy link
Contributor

@thomaszurkan-optimizely thomaszurkan-optimizely left a comment

Choose a reason for hiding this comment

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

LGTM

@thomaszurkan-optimizely thomaszurkan-optimizely merged commit c20c077 into master Oct 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants