Skip to content

feat(backend): basic FGA model#1392

Merged
1 commit merged intoedgehog-device-manager:mainfrom
Dam-99:feat/fga-base-model
May 6, 2026
Merged

feat(backend): basic FGA model#1392
1 commit merged intoedgehog-device-manager:mainfrom
Dam-99:feat/fga-base-model

Conversation

@Dam-99
Copy link
Copy Markdown
Contributor

@Dam-99 Dam-99 commented Apr 23, 2026

What this PR does / why we need it:

Introduce a skeleton for the FGA model, to be expanded when all the requirements are defined.

Additional documentation e.g. usage docs, diagrams, reviewer notes, etc.:

The model in this PR is probably not in its final state, and will likely be reworked in the future, but it's a starting point to add FGA to Edgehog


Thanks for sending a pull request! If this is your first time, here are some tips for you:
  1. You can take a look at our developer guide for an introduction on Edgehog development!
  2. Make sure to read CONTRIBUTING.md and CODE_OF_CONDUCT.md
  3. If the PR is unfinished or you're actively working on it, mark it as draft

When fixing existing issues, use github's syntax to link your pull request to it

fixes #<issue number>

We also have a syntax to signal dependencies to other open pull requests

depends on #<pr number>
depends on https://github.com/...

In case of stacked PRs, you may add the PR number in the last commit's title instead:

gitGraph
    commit id: "Current master"
    branch feat1
    checkout feat1
    commit id: "feat: add something"
    commit id: "feat: add something else (#100)"
    branch feat2
    checkout feat2
    commit id: "refactor: do something"
    commit id: "fix: solve issue"
    commit id: "feat: add a feature (#101)"
    branch feat3
    checkout feat3
    commit id: "feat: feat without pr number"
Loading

@Dam-99 Dam-99 force-pushed the feat/fga-base-model branch 3 times, most recently from 2659ad3 to 05a1ee8 Compare April 23, 2026 09:36
@Dam-99 Dam-99 force-pushed the feat/fga-base-model branch 2 times, most recently from e870eab to 6831af6 Compare May 5, 2026 08:50
@Dam-99 Dam-99 marked this pull request as ready for review May 5, 2026 08:56
Copy link
Copy Markdown
Collaborator

@lusergit lusergit left a comment

Choose a reason for hiding this comment

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

This is a good starting point, we'll need to refine it for sure but it's ok 👍

Copy link
Copy Markdown
Collaborator

@OmarBrbutovic OmarBrbutovic left a comment

Choose a reason for hiding this comment

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

Since the PR had more commits and they were probably squashed, the commit message got a bit messed up, could you please fix it

@Dam-99 Dam-99 force-pushed the feat/fga-base-model branch 2 times, most recently from bc70769 to cd9647e Compare May 6, 2026 12:44
Introduce a basic FGA model to be expanded more thoroughly when all the
requirements are defined.

Signed-off-by: Damiano Mason <damiano.mason@secomind.com>
@Dam-99 Dam-99 force-pushed the feat/fga-base-model branch from cd9647e to 471197e Compare May 6, 2026 12:56
@lusergit lusergit self-assigned this May 6, 2026
@edgehog-app edgehog-app Bot closed this pull request by merging all changes into edgehog-device-manager:main in a40e94b May 6, 2026
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.

3 participants