Skip to content
This repository has been archived by the owner on Oct 23, 2024. It is now read-only.

Feature/entitlements #131

Closed
wants to merge 3 commits into from
Closed

Feature/entitlements #131

wants to merge 3 commits into from

Conversation

MrGuzior
Copy link
Contributor

Checklist

  • Was the code tested in lab?
  • Have you reviewed the code yourself?
  • Are the configs updated?
  • Does the readme need an update?

Write the description and the reason for this PR below ↓

@MrGuzior MrGuzior requested a review from a team as a code owner February 22, 2024 10:28
@MrGuzior MrGuzior closed this Feb 22, 2024
Copy link
Contributor

@MattiasOlla MattiasOlla left a comment

Choose a reason for hiding this comment

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

Vet inte om jag tycker att det här hör hemma i lu-common. Det här känns som konfiguration, det här är mer till för delad logik. Det känns onödigt att behöva uppdatera, bumpa version, skapa en release och sen bumpa versionen i alla tjänster som använder den, bara för att uppdatera plus allt-config. Dessutom är det här repot publikt, och jag vet inte om det här ska vara öppet för världen.

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.

2 participants