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

Add Category Entity #56

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

Conversation

alans-42
Copy link
Collaborator

Proposed changes


Add the entire Category Entity #52

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

emilkovacev and others added 30 commits July 23, 2022 06:51
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???
* 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
Fetching and using the bearer token
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
…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
WeirdAlex03 and others added 23 commits October 14, 2022 16:46
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
@jessehartloff jessehartloff changed the base branch from master to upstream-master April 18, 2023 21:10
@jessehartloff jessehartloff changed the base branch from upstream-master to master April 18, 2023 21:11
@jessehartloff jessehartloff changed the base branch from master to category April 25, 2023 20:07
@jessehartloff jessehartloff changed the base branch from category to main April 25, 2023 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants