Category score #67

jessehartloff · 2023-04-18T21:08:54Z

Proposed changes

Add the Category Score entity to the API. Also merging several dependency and documentation updates.

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)
[ X ] New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ X ] 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.

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

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!

Postman is an invaluable resource for new developers on this api. Having a pre-configured json file for postman will allow developers to understand exactly what a request looks like, and how they can make that request. Another option would be to host that json config file somewhere in this repo, so that we can make updates as we develop new endpoints.

Why is this file in the repo???

Delete .DS_Store

* 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

Updated Schema Documentation

Postman Setup Docs

Fetching and using the bearer token

Minor updates to the schema

This resolves all vulnerabilities reported by `npm audit`, including the breaking change to `passport` from `^0.4.1` to `^0.6.0`. All tests still pass with these updates. Fixes #15

Fixes #18

[Dependencies] Update deps to resolve vulns

Bumps [passport-saml](https://github.com/node-saml/passport-saml) from 3.2.1 to 3.2.2. - [Release notes](https://github.com/node-saml/passport-saml/releases) - [Changelog](https://github.com/node-saml/passport-saml/blob/master/CHANGELOG.md) - [Commits](node-saml/passport-saml@v3.2.1...v3.2.2) --- updated-dependencies: - dependency-name: passport-saml dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]>

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

No idea if this will Just Work (TM), but I guess we'll find out soon!

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

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)

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

…17.3

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

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

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

Category Score Serializer

[GitHub Actions] Fix formatter action

Bumps [minimatch](https://github.com/isaacs/minimatch) from 3.0.4 to 3.1.2. - [Release notes](https://github.com/isaacs/minimatch/releases) - [Changelog](https://github.com/isaacs/minimatch/blob/main/changelog.md) - [Commits](isaacs/minimatch@v3.0.4...v3.1.2) --- updated-dependencies: - dependency-name: minimatch dependency-type: indirect ... 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 [@xmldom/xmldom](https://github.com/xmldom/xmldom) from 0.7.6 to 0.7.10. - [Release notes](https://github.com/xmldom/xmldom/releases) - [Changelog](https://github.com/xmldom/xmldom/blob/master/CHANGELOG.md) - [Commits](xmldom/xmldom@0.7.6...0.7.10) --- updated-dependencies: - dependency-name: "@xmldom/xmldom" dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]>

Updated documentation

Add categoryScore Model

Implement CategoryScore service #57

…en-9.0.0

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

…nimatch-3.1.2 build(deps): bump minimatch from 3.0.4 to 3.1.2

…ldom/xmldom-0.7.10 build(deps): bump @xmldom/xmldom from 0.7.6 to 0.7.10

emilkovacev and others added 30 commits July 23, 2022 06:51

Fetching and using the bearer token

8c3b8a3

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!

Additional authentication setup steps

76ba7bd

Delete .DS_Store

af15826

Why is this file in the repo???

Merge pull request #3 from makeopensource/remove-ds-store

4835cf9

Delete .DS_Store

Added non-implemented schema members

9a379cc

* 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

f94bfb5

fixed markdown checklist syntax

c5284c0

Merge pull request #5 from emilkovacev/master

01e41dc

Updated Schema Documentation

Merge pull request #1 from emilkovacev/postman-setup-docs

e65a160

Postman Setup Docs

Merge pull request #2 from emilkovacev/setup-auth-docs

086614b

Fetching and using the bearer token

Move schema to docs folder; Add released field to AssignmentProblemScore

1a18285

Merge pull request #6 from makeopensource/documentation

b4200ef

Minor updates to the schema

[Dependencies] Update deps to resolve vulns

e451fba

This resolves all vulnerabilities reported by `npm audit`, including the breaking change to `passport` from `^0.4.1` to `^0.6.0`. All tests still pass with these updates. Fixes #15

[Documentation] Fix project referred to as example

a1e5c9a

Fixes #18

docs(README): Capitalize header on line 12

c811186

docs(README): Clean up spelling/formatting errors

7fe5c18

Merge pull request #16 from WeirdAlex03/WeirdAlex03/issue15

cd05c24

[Dependencies] Update deps to resolve vulns

Merge branch 'master' into WeirdAlex03/issue18

77d57c0

ci(GH Actions): Initial basic build & test

6e078f5

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

ab6cf9e

package.json defines requirement node >= 16

ci(GH Actions): Disable fail-fast, new format run

e1445e1

ci(GH Actions): Fix format run

9220a68

ci(GH Actions): Add Docker setup to build and test

43fcf4b

No idea if this will Just Work (TM), but I guess we'll find out soon!

ci(GH Actions): Edit docker run command

cb5c42a

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

ci(GH Actions): Separate test, add annotations

0f47927

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

02d5519

ci(GH Actions): Restructure actions into one file

e675f11

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

a2624cf

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

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

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 #38 from anxiety-fan/master

361c899

Category Score Router Complete (Awaiting Controller and Validator)

Merge pull request #45 from makeopensource/dependabot/npm_and_yarn/qs…

6026dd7

…-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 #46 from makeopensource/dependabot/npm_and_yarn/ex…

0a8a2f5

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

Merge pull request #47 from makeopensource/dependabot/npm_and_yarn/js…

cb35857

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

Merge pull request #48 from makeopensource/dependabot/npm_and_yarn/js…

7ea825f

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

Merge branch 'category-score' into master

46e60df

Merge pull request #49 from zhikangxie107/master

a326f70

Category Score Serializer

Merge pull request #51 from WeirdAlex03/WeirdAlex03/issue43

8a715e7

[GitHub Actions] Fix formatter action

style: run formatter

4594874

style: run formatter

9a8d0c8

Merge pull request #55 from alans-42/doc-branch

e5a6c0d

Updated documentation

Merge pull request #58 from alans-42/catgeoryScore

1554c8c

Add categoryScore Model

Merge pull request #60 from mattrrubino/category-score

2273f40

Implement CategoryScore service #57

Merge branch 'category-score' into dependabot/npm_and_yarn/jsonwebtok…

e5e6f9f

…en-9.0.0

Merge pull request #64 from makeopensource/dependabot/npm_and_yarn/js…

aa031a2

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

Merge pull request #65 from makeopensource/dependabot/npm_and_yarn/mi…

adcc323

…nimatch-3.1.2 build(deps): bump minimatch from 3.0.4 to 3.1.2

Merge pull request #66 from makeopensource/dependabot/npm_and_yarn/xm…

5f89be0

…ldom/xmldom-0.7.10 build(deps): bump @xmldom/xmldom from 0.7.6 to 0.7.10

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

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Category score #67

Category score #67

jessehartloff commented Apr 18, 2023

Category score #67

Are you sure you want to change the base?

Category score #67

Conversation

jessehartloff commented Apr 18, 2023

Proposed changes

Types of changes

Checklist