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

Multitenancy, Filtering Results, CSV Downloads, Copy Surveys, Active/Inactive Surveys #148

Merged
merged 39 commits into from
Jan 2, 2023

Conversation

codeodor
Copy link
Contributor

@codeodor codeodor commented Mar 9, 2021

This PR includes a lot of improvements I've made over time, if you are interested in using them. Some of the commits are outdated, but I recently merged in the most up-to-date main branch, so it should merge cleanly.

All user of the code needs to do is run the migrations and add a method in their `ApplicationController`
Closes codemancers#6
Rapidfire includes a method that looks to allow the programmer to specify `current_admin` or `current_student` or whatever scope of user they want, but it ends up using `current_user` anyway. This commit fixes that issue, and renames the method to something that is obviously in use for rapidfire, as opposed to the general `scoped` name.
@iffyuva
Copy link
Member

iffyuva commented Jan 2, 2023

Thanks for this @codeodor. Will reach out to you in case of any issues

@iffyuva iffyuva merged commit ce76a3e into codemancers:master Jan 2, 2023
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.

None yet

2 participants