Add categoryScore Model #58

alans-42 · 2023-03-28T20:44:45Z

Proposed changes

Add the categoryScore model

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

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 makeopensource#15

Fixes makeopensource#18

…ue15 [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

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

Bumps [@xmldom/xmldom](https://github.com/xmldom/xmldom) from 0.7.5 to 0.7.6. - [Release notes](https://github.com/xmldom/xmldom/releases) - [Changelog](https://github.com/xmldom/xmldom/blob/master/CHANGELOG.md) - [Commits](xmldom/xmldom@0.7.5...0.7.6) --- updated-dependencies: - dependency-name: "@xmldom/xmldom" dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]>

…pm_and_yarn/xmldom/xmldom-0.7.6 Bump @xmldom/xmldom from 0.7.5 to 0.7.6

…ue18 [Documentation] Fix project referred to as example

…pm_and_yarn/passport-saml-3.2.2 Bump passport-saml from 3.2.1 to 3.2.2

Add GitHub user jessehartloff as a code owner with the authority to a…

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

[GitHub Actions] Set up GitHub Actions to automatically build, test, and format

- Added postman file config - Edited docs for postman file

Revised Postman Documentation

Removed the letter grade enum from the model

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 makeopensource#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 makeopensource#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 makeopensource#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 makeopensource#15

[Documentation] Fix project referred to as example

a1e5c9a

Fixes makeopensource#18

docs(README): Capitalize header on line 12

c811186

docs(README): Clean up spelling/formatting errors

7fe5c18

Merge pull request makeopensource#16 from WeirdAlex03/WeirdAlex03/iss…

cd05c24

…ue15 [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 22 commits October 14, 2022 16:46

ci(GH Actions): Clean up action runs

16e4c42

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

ddc6ce6

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

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

eeb7386

…pm_and_yarn/xmldom/xmldom-0.7.6 Bump @xmldom/xmldom from 0.7.5 to 0.7.6

Merge pull request makeopensource#19 from WeirdAlex03/WeirdAlex03/iss…

fe8665d

…ue18 [Documentation] Fix project referred to as example

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

7254b05

…pm_and_yarn/passport-saml-3.2.2 Bump passport-saml from 3.2.1 to 3.2.2

Merge pull request makeopensource#26 from UBAutograding/master

052d51b

Add GitHub user jessehartloff as a code owner with the authority to a…

ci(GH Actions): Formatter needs to commit results

46c88c0

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

3e7a255

ci(GH Actions): Try different method to commit

5412e59

Merge 5412e59 into b4200ef

c2e2b25

style: run formatter

2f80b8a

ci(GH Actions): Different commit technique

a2f07be

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

0c35121

style: run formatter

bec0ff1

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

c84da63

This reverts the failing test intentionally added in ddc6ce6

ci(GH Actions): Formatter needs to commit results

66084d4

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

Merge branch 'master' into issue22

20e571a

Merge pull request makeopensource#23 from WeirdAlex03/issue22

641e9da

[GitHub Actions] Set up GitHub Actions to automatically build, test, and format

Added to docs

b422558

- Added postman file config - Edited docs for postman file

Merge pull request makeopensource#42 from emilkovacev/master

a59642b

Revised Postman Documentation

Add categoryScore Model

8a21ad3

alans-42 requested a review from jessehartloff as a code owner March 28, 2023 20:44

jessehartloff changed the base branch from master to category-score March 28, 2023 21:05

Removed Letter Grade

e258d00

Removed the letter grade enum from the model

jessehartloff merged commit 1554c8c into makeopensource:category-score Apr 18, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add categoryScore Model #58

Add categoryScore Model #58

alans-42 commented Mar 28, 2023

Add categoryScore Model #58

Add categoryScore Model #58

Conversation

alans-42 commented Mar 28, 2023

Proposed changes

Types of changes

Checklist