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.

Sign up for GitHub

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

Jump to bottom

Board controller authorizations #79

Merged
merged 24 commits into from
Nov 7, 2023
Merged

Board controller authorizations #79

merged 24 commits into from
Nov 7, 2023

Conversation

akinsey
Copy link
Member

@akinsey akinsey commented Oct 31, 2023

No description provided.

unenglishable
unenglishable requested changes Nov 2, 2023
View reviewed changes
Copy link
Member

@unenglishable unenglishable left a comment

Choose a reason for hiding this comment

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

add tests

unenglishable and others added 18 commits November 2, 2023 14:24
@unenglishable
test(seed+conn_case): seed private user and prepare in conn_case
54ee774 
can be used for checking acl's (private user can only access motd)
@unenglishable
test(controllers/board): check by_category errors on invalid permissions
f2364d5
@unenglishable
test(controllers/board): check invalid board read access for unauthen…
12f4d80 
…ticated user
@akinsey
test(seed/users): add seed for every role type
3ce8b88
@akinsey
test(conn_case): add all new roles from user seed to conn case
8d43e8e
@akinsey
test(session_test): updating tests with banned tag to use banned user
88ffd0c
@akinsey
test(models/mention): update mention test to user private role instea…
db74461 
…d of banned to test invalid permissions
@akinsey
style(format): run mix format
59d9649
@unenglishable
test(controllers/board): update unauthenticated test description
241e510
@unenglishable
test(controllers/board): check board find when authenticated
46e87d0
@unenglishable
test(controllers/board): check board find when authenticated as admin
95ed14e
@unenglishable
test(controllers/board): test super admin board find
55a1e6c
@unenglishable
Merge remote-tracking branch 'origin/main' into board-controller-auth…
17ba087 
…orizations-tests
@unenglishable
Merge remote-tracking branch 'origin/board-controller-authorizations'…
4d85649 
… into board-controller-authorizations-tests

resolve conflicts
@unenglishable
test(session): use new banned role in test
57bc64d
@unenglishable
test(controllers/board): mix format
5855fd9
@unenglishable
test(support/conn_case): fix merge
fe38cd6
@akinsey
Merge pull request #83 from epochtalk/board-controller-authorizations…
0564914 
…-tests

Board controller authorizations tests
unenglishable
unenglishable approved these changes Nov 7, 2023
View reviewed changes
Copy link
Member

@unenglishable unenglishable left a comment

Choose a reason for hiding this comment

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

👍

@unenglishable unenglishable merged commit 4dfd4fb into main Nov 7, 2023
5 checks passed
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 7, 2023

🎉 This PR is included in version 1.23.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@unenglishable unenglishable deleted the board-controller-authorizations branch November 8, 2023 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@unenglishable unenglishable unenglishable approved these changes

Assignees

@unenglishable unenglishable

Labels
released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@akinsey @unenglishable