Added AssignmentScore Entity #63

zhikangxie107 · 2023-04-18T20:18:45Z

Proposed changes

Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.

Types of changes

What types of changes does your code introduce?
Put an x in the boxes that apply

Bugfix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Documentation Update (if none of the other choices apply)

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. This is simply a reminder of what we are going to look for before merging your code.

My changeset covers only what is described above (no extraneous changes)
Lint and unit tests pass locally with my changes
I have added tests that prove my fix is effective or that my feature works
I have added necessary documentation (if appropriate)
Any dependent changes have been merged downstream

Further comments

If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Add the submission problem score entity Add documentation rename filenames to be consistently singular Co-authored-by: Emil Kovacev <[email protected]> Co-authored-by: Emil Kovacev <[email protected]> Co-authored-by: John Abramo <[email protected]> Co-authored-by: Alex M <[email protected]> Co-authored-by: dylanzinsley <[email protected]> Co-authored-by: Adrian Carter <[email protected]> Co-authored-by: alan soto <[email protected]> Co-authored-by: Alan Soto <[email protected]> Co-authored-by: --global <--global> Co-authored-by: Adrian Carter <[email protected]> Co-authored-by: zhikangxie107 <[email protected]> Co-authored-by: Dylanz5 <[email protected]> Co-authored-by: ToNewAutumn <[email protected]>

Add code assignment entity Integrate MinIO as a solution for saving files to disk

Add submissionScore entity to the API. This entity stores a score and feedback attached to a single submission

Bumps [qs](https://github.com/ljharb/qs) to 6.11.0 and updates ancestor dependencies [qs](https://github.com/ljharb/qs), [body-parser](https://github.com/expressjs/body-parser) and [express](https://github.com/expressjs/express). These dependencies need to be updated together. Updates `qs` from 6.7.0 to 6.11.0 - [Release notes](https://github.com/ljharb/qs/releases) - [Changelog](https://github.com/ljharb/qs/blob/main/CHANGELOG.md) - [Commits](ljharb/qs@v6.7.0...v6.11.0) Updates `body-parser` from 1.19.0 to 1.20.1 - [Release notes](https://github.com/expressjs/body-parser/releases) - [Changelog](https://github.com/expressjs/body-parser/blob/master/HISTORY.md) - [Commits](expressjs/body-parser@1.19.0...1.20.1) Updates `express` from 4.17.1 to 4.18.2 - [Release notes](https://github.com/expressjs/express/releases) - [Changelog](https://github.com/expressjs/express/blob/master/History.md) - [Commits](expressjs/express@4.17.1...4.18.2) --- updated-dependencies: - dependency-name: qs dependency-type: indirect - dependency-name: body-parser dependency-type: direct:production - dependency-name: express dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]>

Bumps [express](https://github.com/expressjs/express) from 4.17.1 to 4.17.3. - [Release notes](https://github.com/expressjs/express/releases) - [Changelog](https://github.com/expressjs/express/blob/master/History.md) - [Commits](expressjs/express@4.17.1...4.17.3) --- updated-dependencies: - dependency-name: express dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]>

Bumps [jsonwebtoken](https://github.com/auth0/node-jsonwebtoken) from 8.5.1 to 9.0.0. - [Release notes](https://github.com/auth0/node-jsonwebtoken/releases) - [Changelog](https://github.com/auth0/node-jsonwebtoken/blob/master/CHANGELOG.md) - [Commits](auth0/node-jsonwebtoken@v8.5.1...v9.0.0) --- updated-dependencies: - dependency-name: jsonwebtoken dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]>

Bumps [json5](https://github.com/json5/json5) from 2.2.0 to 2.2.3. - [Release notes](https://github.com/json5/json5/releases) - [Changelog](https://github.com/json5/json5/blob/main/CHANGELOG.md) - [Commits](json5/json5@v2.2.0...v2.2.3) --- updated-dependencies: - dependency-name: json5 dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]>

Update CODEOWNERS (makeopensource#45)

* Added a menu for the schema members * Added comments for new members * I suggest we replace comments with types, since "foreign key" seems easier to follow

Every time I want to start up this project, I spend 30 or so minutes figuring out authorization. Instead, I figured it would be easier to put an example in the documentation to cut setup time and enable more commits in the future!

Pretty much just default Node.js config, just changed to only run on PRs (instead of pushes and PRs targetting "master")

package.json defines requirement node >= 16

GH concats it all into one line so escaping the line breaks isnt needed

This uses incorporates mattallty/jest-github-action to add annotations in the test files on the failing tests, making it easier to identify the problems

Test and Format now rely on Build, reducing duplicate code (if this works the way I think it will)

Last commit: > Test and Format now rely on Build, reducing duplicate code > (if this works the way I think it will) It did not. They're still unique environments, so it still has to be set up for each action

Intentionally failing a test and messing up file formatting to see how the actions deal with that

I won't tell it to actually commit yet, first I'll review the output to make sure it isn't committing too much

This reverts the failing test intentionally added in ddc6ce6

I won't tell it to actually commit yet, first I'll review the output to make sure it isn't committing too much

- Added postman file config - Edited docs for postman file

# Conflicts: # README.md # package-lock.json # package.json

Category Score Router Complete (Awaiting Controller and Validator)

…pm_and_yarn/qs-and-body-parser-and-express-6.11.0 build(deps): bump qs, body-parser and express

…17.3

…pm_and_yarn/express-4.17.3 build(deps): bump express from 4.17.1 to 4.17.3

…pm_and_yarn/jsonwebtoken-9.0.0 build(deps): bump jsonwebtoken from 8.5.1 to 9.0.0

…pm_and_yarn/json5-2.2.3 build(deps): bump json5 from 2.2.0 to 2.2.3

zhikangxie107 and others added 22 commits October 19, 2022 17:18

added submission problem score file to controller

175121c

add test for SubmissionProblemScore

f0a2892

Renamed submissionproblemscore test file

29dbc56

fix spelling errors and indentation

7f3b59a

Merge branch 'makeopensource:master' into master

3b3c108

added serializer for categoryScore

0019cd5

Category Score Router: singular

2521bcc

CodeAssignment entity + MinIO integration (makeopensource#28)

ee8185b

Add code assignment entity Integrate MinIO as a solution for saving files to disk

Submission score entity (makeopensource#44)

52188cb

Add submissionScore entity to the API. This entity stores a score and feedback attached to a single submission

categoryScore.serializer.test

beb5f4d

Merge branch 'master' of https://github.com/zhikangxie107/devU-api

5793137

added Model for Assignment Score

4d503ee

added Model for AssignmentScore

d89c957

implemented AssignmentScore Controller

2da107f

Update CODEOWNERS (makeopensource#45)

7ece6d7

Merge pull request makeopensource#61 from UBAutograding/master

3c5e9d4

Update CODEOWNERS (makeopensource#45)

controller, router, service, serializer, validator

9fd67c3

zhikangxie107 requested a review from jessehartloff as a code owner April 18, 2023 20:18

emilkovacev and others added 7 commits April 18, 2023 16:25

Added non-implemented schema members

ff24fa0

* Added a menu for the schema members * Added comments for new members * I suggest we replace comments with types, since "foreign key" seems easier to follow

added types to schema doc

30171fc

fixed markdown checklist syntax

6bd5871

Fetching and using the bearer token

12fdb18

Every time I want to start up this project, I spend 30 or so minutes figuring out authorization. Instead, I figured it would be easier to put an example in the documentation to cut setup time and enable more commits in the future!

Move schema to docs folder; Add released field to AssignmentProblemScore

6a151c5

ci(GH Actions): Initial basic build & test

c724b20

Pretty much just default Node.js config, just changed to only run on PRs (instead of pushes and PRs targetting "master")

ci(GH Actions): Remove Node v14 from tests

357b3e0

package.json defines requirement node >= 16

WeirdAlex03 and others added 27 commits April 18, 2023 16:25

ci(GH Actions): Edit docker run command

1b699e0

GH concats it all into one line so escaping the line breaks isnt needed

ci(GH Actions): Separate test, add annotations

8e5e03c

This uses incorporates mattallty/jest-github-action to add annotations in the test files on the failing tests, making it easier to identify the problems

ci(GH Actions): Fix test action

6ba1398

ci(GH Actions): Restructure actions into one file

e00b463

Test and Format now rely on Build, reducing duplicate code (if this works the way I think it will)

ci(GH Actions): Fix actions again

9168869

Last commit: > Test and Format now rely on Build, reducing duplicate code > (if this works the way I think it will) It did not. They're still unique environments, so it still has to be set up for each action

ci(GH Actions): Clean up action runs

83ce05a

test(GH Actions): Ex. of failing test, formatting

3e71bf1

Intentionally failing a test and messing up file formatting to see how the actions deal with that

ci(GH Actions): Formatter needs to commit results

497f602

I won't tell it to actually commit yet, first I'll review the output to make sure it isn't committing too much

ci(GH Actions): Looks good, actually committing

e4ee9b9

ci(GH Actions): Try different method to commit

290f44e

style: run formatter

b8ca75a

ci(GH Actions): Different commit technique

948fccd

ci(GH Actions): Fix checkout to work with commit

2627f49

style: run formatter

e1d0cb2

test(GH Actions): It works, undo broken test

c6296a1

This reverts the failing test intentionally added in ddc6ce6

ci(GH Actions): Formatter needs to commit results

27eee89

I won't tell it to actually commit yet, first I'll review the output to make sure it isn't committing too much

docs(README): Clean up spelling/formatting errors

fc7f6e8

Added to docs

f92e1ba

- Added postman file config - Edited docs for postman file

Merge remote-tracking branch 'origin/master' into category-score

525b483

# Conflicts: # README.md # package-lock.json # package.json

Merge branch 'category-score' into master

569177a

Merge pull request makeopensource#38 from anxiety-fan/master

361c899

Category Score Router Complete (Awaiting Controller and Validator)

Merge pull request makeopensource#45 from makeopensource/dependabot/n…

6026dd7

…pm_and_yarn/qs-and-body-parser-and-express-6.11.0 build(deps): bump qs, body-parser and express

Merge branch 'category-score' into dependabot/npm_and_yarn/express-4.…

8d956f2

…17.3

Merge pull request makeopensource#46 from makeopensource/dependabot/n…

0a8a2f5

…pm_and_yarn/express-4.17.3 build(deps): bump express from 4.17.1 to 4.17.3

Merge pull request makeopensource#47 from makeopensource/dependabot/n…

cb35857

…pm_and_yarn/jsonwebtoken-9.0.0 build(deps): bump jsonwebtoken from 8.5.1 to 9.0.0

Merge pull request makeopensource#48 from makeopensource/dependabot/n…

7ea825f

…pm_and_yarn/json5-2.2.3 build(deps): bump json5 from 2.2.0 to 2.2.3

Merge branch 'category-score' into master

46e60df

jessehartloff changed the base branch from master to upstream-master April 18, 2023 21:11

jessehartloff requested a review from NicholasMy as a code owner April 18, 2023 21:11

jessehartloff changed the base branch from upstream-master to master April 18, 2023 21:11

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Added AssignmentScore Entity #63

Added AssignmentScore Entity #63

zhikangxie107 commented Apr 18, 2023

Added AssignmentScore Entity #63

Are you sure you want to change the base?

Added AssignmentScore Entity #63

Conversation

zhikangxie107 commented Apr 18, 2023

Proposed changes

Types of changes

Checklist

Further comments