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

Polls #91

Merged
merged 45 commits into from
Mar 19, 2024
Merged

Polls #91

merged 45 commits into from
Mar 19, 2024

Conversation

akinsey
Copy link
Member

@akinsey akinsey commented Mar 5, 2024 •
edited
Loading

Poll create, will come back and implement poll update and delete

akinsey added 30 commits November 27, 2023 17:22
@akinsey
refactor(models/post): wip implement post update db function and chan…
cc2b888 
…geset
@akinsey
refactor(models/post): wip implement post update
09994a7
@akinsey
feat(models/post): implement post update
8daf839
@akinsey
refactor(controllers/post): wip implement post update route
9928324
@akinsey
fix(models/post): update needs to preload thread
97e86d5
@akinsey
refactor(controller/posts): wip implementMentions hooks for post update
10d4a92
@akinsey
feat(port-roadmap): update roadmap with hooks for post update
ed14155
@akinsey
refactor(controllers/post): add todo completetion list for post update
00b880f
@akinsey
style(format): run mix format
1e3562c
@akinsey
refactor(models/post): update changeset modified to pass credo
0c63e73
@akinsey
refactor(controllers/post): wip add authorizations for post update route
393066a
@akinsey
feat(models/post): add is_locked function for post update authorizations
c1b5e91
@akinsey
refactor(controllers/post): wip implement authorizations for post and…
18cc0c6 
… thread lock for post update
@akinsey
refactor(controllers/post): wip implement has_priority function
21c3d1c
@akinsey
fix(controllers/post): has_priority should not reject directly, retur…
971dd1c 
…n false if user doesnt have permission
@akinsey
refactor(controllers/post): wip authorizations, finish has_priority f…
5845ed0 
…unction
@akinsey
refactor(models/post): add option to preload post authors roles
2d3163b
@akinsey
refactor(controllers/post): update http error codes to be correct for…
20a1228 
…bidden code
@akinsey
refactor(controllers/post): implement bypass owner check for post update
3e93717
@akinsey
refactor(models/post): remove unneeded is_locked db query in Post model
b226050
@akinsey
refactor(controller/post): add authorization to check if post is dele…
6863163 
…ted/hidden on post update
@akinsey
refactor(controllers/post): wip implement authorizations, misc refactor
3c1b5f7
@akinsey
feat(models/board): implement db function that can check if post is s…
f67717e 
…till editable given thread id and post created at
@akinsey
style(format): run mix format
efb59d8
@akinsey
fix(controllers/post): refactor authorizations and fix permission che…
eedd6f7 
…ck for is_mod related authorizations
@akinsey
refactor(controllers/post): resolve credo issues
64e42a3
@akinsey
refactor(controllers/post): code to be more readable
e692f80
@akinsey
feat(helpers/acl): add docs and spec for acl has role function
ab1dde9
@akinsey
fix(json/post): resolve issue with error being thrown after editing a…
aa18a8c 
… post
@akinsey
fix(models/board): issue with dialyzer having no return in post updat…
be8ae24 
…e route
akinsey added 12 commits December 21, 2023 14:02
@akinsey
feat(models/pollresponse): implement create for poll response model, …
0a5d868 
…used for voting
@akinsey
fix(models/poll_response): add repo alias
991f7ae
@akinsey
feat(models/poll): implement has_voted database function, refactor po…
415cb36 
…llresponse create
@akinsey
refactor(models/poll_response): update poll response create to take i…
a5522ca 
…n attrs as param
@akinsey
refactor(models/poll_response): update poll response to take string a…
52d17a1 
…nswer_id
@akinsey
feat(models/poll): update ecto model and derive in preperation for re…
18b158b 
…turning json object from vote api route
@akinsey
refactor(models/poll_answer): update derive for rendering poll json
4089161
@akinsey
feat(controllers/thread): implement vote api route
bce05cd
@akinsey
feat(json/thread): update vote json file for api data rendering, add …
5f03b58 
…route for voting to router
@akinsey
fix(misc): for passing code testing and formatting
0ba67f0
@akinsey
Merge remote-tracking branch 'origin/main' into polls
560b6c5
@akinsey
feat(update-deps): update deps
3a92a6d
@akinsey akinsey requested a review from unenglishable March 5, 2024 19:58
unenglishable
unenglishable approved these changes Mar 19, 2024
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.

taking note to review "is_" boolean method standards in the future
approved 🦖

@unenglishable unenglishable merged commit bff9fa7 into main Mar 19, 2024
5 checks passed
@github-actions GitHub Actions
Copy link

🎉 This PR is included in version 1.27.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@crod951 crod951 deleted the polls branch February 19, 2025 17:36
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
No one assigned
Labels
released
Projects
None yet
Milestone
No milestone
Development

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