Skip to content

Add basic frontend for displaying list of feature flags on front page…#9

Merged
lucidrains merged 1 commit into
masterfrom
pw/add-flag-creation-crud
Oct 6, 2017
Merged

Add basic frontend for displaying list of feature flags on front page…#9
lucidrains merged 1 commit into
masterfrom
pw/add-flag-creation-crud

Conversation

@lucidrains
Copy link
Copy Markdown
Contributor

…, with ability to click into a feature flag for additional information

Copy link
Copy Markdown
Collaborator

@zhouzhuojie zhouzhuojie left a comment

Choose a reason for hiding this comment

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

Left one comment about the :id route. LGTM

Comment thread browser/flagr-ui/src/router/index.js Outdated
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

super nit, s/:id/:flagID, may want to keep it the same as the definition of swagger.yml

…, with ability to click into a feature flag for additional information
@lucidrains lucidrains force-pushed the pw/add-flag-creation-crud branch from 554f252 to 8fba299 Compare October 6, 2017 03:52
@lucidrains lucidrains merged commit 71e88fc into master Oct 6, 2017
@lucidrains lucidrains deleted the pw/add-flag-creation-crud branch October 6, 2017 03:53
iamafanasyev pushed a commit to iamafanasyev/flagr that referenced this pull request Feb 5, 2026
Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3 to 4.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@v3...v4)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-version: '4'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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