Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Kunna ta bort access för specifika poster/personer #131

Open
julialovisakarlsson opened this issue Jan 30, 2024 · 5 comments · May be fixed by #350
Open

Kunna ta bort access för specifika poster/personer #131

julialovisakarlsson opened this issue Jan 30, 2024 · 5 comments · May be fixed by #350
Assignees
Labels
enhancement New feature or request

Comments

@julialovisakarlsson
Copy link

julialovisakarlsson commented Jan 30, 2024

Kan vara bra att kunna ta bort access till specifik dörr om någon misskött sig, utan att behöva ta bort access för hela personens post. Dessutom nice om det kunde vara tidsbestämt.

Alternativt om en dörr borde vara öppen för ett helt utskott, förutom en tillhörande post av någon anledning, nice att bara fylla i att utskottet ska ha, och lägga till att den specifika posten inte ska ha.

Tasks

No tasks being tracked yet.
@fnurkla fnurkla added the enhancement New feature or request label Jan 30, 2024
@fnurkla
Copy link
Contributor

fnurkla commented Jan 30, 2024

fulaccess
fulaccess2-small

@Joel-Liden Joel-Liden self-assigned this Feb 6, 2024
@Joel-Liden Joel-Liden assigned Joel-Liden and unassigned Joel-Liden Apr 16, 2024
@Joel-Liden
Copy link
Collaborator

Joel-Liden commented May 7, 2024

Gjort

    • Möjligt att lägga till om en access är en ban eller en access till en dörr. Detta kontrolleras genom en toggle knapp.
  • Accesser är röda ifall de är bans.
  • En beskrivning kan publiceras tillsammans med en ban (eller access), men det är frivilligt.
  • Definiera om funktionaliteten av en doorAccess så att den beror på att den finns en access under perioden samtidigt som det inte finns en ban access.
  • Fixa beskrivningen så att det visas i en pop-up
  • snyggare mer mobilvänlig ui
  • Text ska läggas bredvid, inte ovanför, ban button.

TODO:
-bug-testning
-smådetaljer

@Joel-Liden Joel-Liden linked a pull request May 14, 2024 that will close this issue
@Joel-Liden
Copy link
Collaborator

Joel-Liden commented May 20, 2024

TODO:

  • Ta bort extra knapp - Done
  • Översättning på knapp - Done (banknapp kvar)
  • Färger
  • Rundningar på knapp (join)
  • Knapp för info custom icon, ej svg - Done
  • Titel på informationsnotifikation - ej titeln på dörren - Fixat
  • För kort table header - Done
  • Övriga översättningar - Notifikation, titlar - Ej påbörjat
  • Bugfixa dörraccess.
    En person kan bannas lätt. Ett roll kan bannas lätt.
    Ifall en person är en medlem i ett utskott med access får den access igen, bugg - Done.
    Ifall en person får access men utskottet är bannat får access igen, bugg.

@Joel-Liden
Copy link
Collaborator

Joel-Liden commented May 31, 2024

Bug possibilites:

Ban a role which already has access from the role ? - Works
Ban a member which already has access from a member ? - Works
Ban a role which already has members with access from members ?
Ban a role which already has members with access from other roles and members ?
Ban a member which already has access from another role ? - works

Ban priority should always go above access prority.

@Joel-Liden
Copy link
Collaborator

Joel-Liden commented Oct 15, 2024

Update:

  • _ All functionality is implemented._
  • Daniel is implementing random generating database for easier bugtesting.

After the functionality is verified, hopefully by someone who knows how this works a bit better, then it can be brought up to date with main.

And finally merged!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 🏗 In Progress
Development

Successfully merging a pull request may close this issue.

3 participants