-
Notifications
You must be signed in to change notification settings - Fork 0
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
Company request schema #111
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cooper to the 🌔 🚀
@@ -0,0 +1,47 @@ | |||
import { relations, sql } from "drizzle-orm"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To get the linter to pass you'll just need to remove sql
from the import statement
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
id: true, | ||
createdAt: true, | ||
}); | ||
// Example TypeScript type for the Company Request |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
super nit pick (i.e. don't have to change) but I don't know what this comment means 💀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yea i dont either
@@ -0,0 +1,48 @@ | |||
import { relations, sql } from "drizzle-orm"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here for the sql
thing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
})); | ||
|
||
/* | ||
What is the relationship between companyRequest/roleRequest/company/role |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is perfect!
- Each company request is attached to a profile (and it also includes the role parameters in the schema)
- Each role request is attached to a company and a profile!
So we can probs remove these comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bet
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
Created a company request schema and role request schema.
Admin should be able to approve new introduced company and roles, therefore we are introducing a new company and role request schema. The schema has all the information needed to create a company/role, profile that request the new role, time it was requested, and the status of the request.
Closes #109
How has this been tested?
haha
Screenshots (if appropriate):
Types of changes
Checklist: