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

E2401 Implementing and testing import & export controllers #2801

Open
wants to merge 61 commits into
base: main
Choose a base branch
from

Conversation

dlfranks
Copy link

Please set the title of your pull request in the following format: [Project id/Independent study/Issue fix/...]. project name, eg., E904. Integrating courses and assignments on Expertiza with Moodle

We suggest you including the below information in your pull request:

  • A description of the changes proposed in the pull request.
  • @mentions of the person or team responsible for reviewing proposed changes.

About the Expertiza Bot

  • If you have any questions about comments given by the expertiza-bot (example), you could create a new comment in your pull request with the content /dispute [UUID1] [UUID2] (example), then the professor and TAs will be notified.
  • Here is a video about how to communicate to the expertiza-bot.

mpatel029 and others added 30 commits March 25, 2024 00:35
Conflicts:
	app/controllers/import_file_controller.rb
	spec/controllers/import_file_controller_spec.rb
…late_key_chart_information, calculate_formatted_percentage
app/controllers/import_file_controller.rb Fixed Show fixed Hide fixed
app/controllers/import_file_controller.rb Fixed Show fixed Hide fixed
app/controllers/import_file_controller.rb Fixed Show fixed Hide fixed
app/controllers/import_file_controller.rb Fixed Show fixed Hide fixed
app/controllers/import_file_controller.rb Fixed Show fixed Hide fixed
app/controllers/import_file_controller.rb Fixed Show fixed Hide fixed
app/controllers/import_file_controller.rb Fixed Show fixed Hide fixed
app/controllers/import_file_controller.rb Fixed Show fixed Hide fixed
app/controllers/export_file_controller.rb Fixed Show fixed Hide fixed
CourseParticipant.import(row_hash, session, params[:id])
end
model = params[:model]
contents_hash = eval(params[:contents_hash])

Check failure

Code scanning / CodeQL

Code injection Critical

This code execution depends on a
user-provided value
.
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

3 participants