Skip to content

feat: remove the unimplemented flag from UpdateStore request#226

Draft
rhamzeh wants to merge 1 commit into
mainfrom
feat/remove-unimplemented-flag-from-update-store
Draft

feat: remove the unimplemented flag from UpdateStore request#226
rhamzeh wants to merge 1 commit into
mainfrom
feat/remove-unimplemented-flag-from-update-store

Conversation

@rhamzeh

@rhamzeh rhamzeh commented May 9, 2025

Copy link
Copy Markdown
Member

Description

This removes the unimplemented flag from the UpdateStore method.

Note: I would prefer that this becomes a PUT instead of a PATCH as it currently is, however will leave that up for discussion before contributing that change

References

Goes in tandem with:

Review Checklist

  • I have clicked on "allow edits by maintainers".
  • I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
  • The correct base branch is being used, if not main
  • I have added tests to validate that the change in functionality is working as expected

@rhamzeh rhamzeh force-pushed the feat/remove-unimplemented-flag-from-update-store branch from 1466469 to f2d3c0d Compare May 9, 2025 00:20
@curfew-marathon

Copy link
Copy Markdown

This PR has had no human activity for 90 days, so it has been marked lifecycle/stale.

This is automated backlog grooming, not a judgment on the work.

What happens next, unless there is activity:

  • In 30 days, it will move to lifecycle/closing-soon
  • 30 days after that, it will be closed

To keep it open, push a commit or leave a comment, and the clock will reset. For work that should not auto-close, such as an RFC or long-running experiment, ask a maintainer to add lifecycle/frozen.

@curfew-marathon curfew-marathon added the lifecycle/stale No human activity in 90 days label Jul 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lifecycle/stale No human activity in 90 days

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants