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

add module anonymous proposals #22

Merged
merged 2 commits into from
Mar 29, 2023
Merged

Conversation

eliegaboriau
Copy link
Contributor

Add module to gemfile
Translate locales to french
Rollback to 0.24 the proposal controller and the proposal model to make it compatible with the module

This module enables the creation of proposal by users not signed in, and set the author to an anonymous user.
Don't forget to add the anonymous user to test it, as it's mentionned in the readme module

@eliegaboriau eliegaboriau requested review from paulinebessoles, a team and Quentinchampenois and removed request for a team March 21, 2023 11:21
@eliegaboriau eliegaboriau marked this pull request as ready for review March 21, 2023 12:22
@paulinebessoles paulinebessoles requested a review from a team March 24, 2023 13:56
@Quentinchampenois Quentinchampenois merged commit 11572d8 into develop Mar 29, 2023
@Quentinchampenois Quentinchampenois deleted the add_anonymous_proposals branch March 29, 2023 15:49
armandfardeau added a commit that referenced this pull request Apr 4, 2023
* Update rack_attack.rb (#21)

* add module anonymous proposals (#22)

* add module and bump already existing files to make it work

* fix specs

* Bump ruby from 2.7.1 to 2.7.5 (#25)

* Reconfigure rack attack (#24)

* Add rack attack configuration

* Add rack_attack

* Fix rubocop offense

* Disable Rack::Attack for specs

* normalize keys

* Ignore Unused

---------

Co-authored-by: Armand Fardeau <[email protected]>
Co-authored-by: eliegaboriau <[email protected]>
Co-authored-by: Quentin Champenois <[email protected]>
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