Skip to content

Conversation

chrisllontop
Copy link
Member

No description provided.

@chrisllontop chrisllontop mentioned this pull request Nov 26, 2024
20 tasks
@chrisllontop chrisllontop requested a review from osbornm November 27, 2024 02:25
@chrisllontop chrisllontop changed the title Implement fallback mechanism for multiple server URLs feat: add OpenFeature provider, tests and examples Nov 28, 2024
@chrisllontop chrisllontop requested a review from osbornm November 28, 2024 22:53
@chrisllontop chrisllontop requested a review from osbornm December 2, 2024 17:27
Copy link
Member

@osbornm osbornm left a comment

Choose a reason for hiding this comment

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

just a couple small things with the examples

Comment on lines 44 to 52
app.get('/feature-flag', async (req, res) => {
const client = OpenFeature.getClient();
const flagDetails = await client.getBooleanDetails('example-feature', false, req.context);

res.json({
featureFlag: 'example-feature',
value: flagDetails.value,
});
});
Copy link
Member

Choose a reason for hiding this comment

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

this works but we'd want something that's more if the flag is true do this logic and false do this other... perhaps something that prints out a message about version or something

@chrisllontop chrisllontop requested a review from osbornm December 2, 2024 22:04
@chrisllontop chrisllontop merged commit 73ed122 into main Dec 2, 2024
@chrisllontop chrisllontop deleted the feature/multi-url branch December 2, 2024 22:06
github-actions bot pushed a commit that referenced this pull request Dec 19, 2024
# 1.0.0 (2024-12-19)

### Bug Fixes

* **hyphenClient:** update horizon server URL to use evaluate endpoint ([c9e2f2d](c9e2f2d))
* **hyphenClient:** update mock horizon URL references to use base URL ([27cb479](27cb479))
* make logger parameter optional in provider functions ([#11](#11)) ([ed1d019](ed1d019))
* Throw error on missing or invalid evaluation ([#10](#10)) ([4e2f24b](4e2f24b))
* typo in CONTRIBUTING.md ([c7a0983](c7a0983))

### Features

* add cache config options and key generation ([#12](#12)) ([1c4fb46](1c4fb46))
* add OpenFeature provider, tests and examples ([#4](#4)) ([73ed122](73ed122))
* add telemetry support ([#15](#15)) ([d1d76d9](d1d76d9))
Copy link

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants