-
Notifications
You must be signed in to change notification settings - Fork 20
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
External mentee visibility policy #625
Open
Prajwal17Tunerlabs
wants to merge
26
commits into
develop
Choose a base branch
from
external-mentee-visibility-policy
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 25 commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
371d125
removed 10mins delete restrictions
nevil-mathew aaf6fb7
Merge pull request #614 from ELEVATE-Project/delete-restrictions
rakeshSgr 647200b
mentee list org policy issue
adithyadinesh0412 b4431a9
Merge pull request #616 from adithyadinesh0412/notify-user-in-sessions
rakeshSgr b7a0456
mentee list org policy issue
adithyadinesh0412 4a2aac7
Merge pull request #618 from adithyadinesh0412/notify-user-in-sessions
rakeshSgr b7511f2
new migrations added
Prajwal17Tunerlabs e234159
added 2 new columns
Prajwal17Tunerlabs 7816a4f
Revert "new migrations added"
Prajwal17Tunerlabs a107788
Revert "added 2 new columns"
Prajwal17Tunerlabs f7ccf50
added new migrations
Prajwal17Tunerlabs 3d6054a
filter list policy update
adithyadinesh0412 bf03186
Merge pull request #620 from adithyadinesh0412/notify-user-in-sessions
rakeshSgr da533c2
Frontend requested form changes
adithyadinesh0412 58471c2
Merge pull request #621 from adithyadinesh0412/notify-user-in-sessions
rakeshSgr e008854
added input validation for mentor profile update
nevil-mathew 38e8b90
updated mentee response message
nevil-mathew dfc0d6a
Merge pull request #622 from ELEVATE-Project/req-body-validation
joffinjoy 2b5b99f
made [org-admin/setOrgPolicies & profile/create] API changes
Prajwal17Tunerlabs 01e2bca
Merge branch 'develop' of github.com:ELEVATE-Project/mentoring into e…
Prajwal17Tunerlabs 3bda094
changes in [profile/filterList] API
Prajwal17Tunerlabs 584f72a
changes in [mentees/list] API
Prajwal17Tunerlabs a44bd36
changes in [org-admin/roleChange] API
Prajwal17Tunerlabs b9941d1
changes in [sessions/update] API
Prajwal17Tunerlabs 53cea65
visibility policy changes
Prajwal17Tunerlabs 4b78037
Merge branch 'develop' of github.com:ELEVATE-Project/mentoring into e…
Prajwal17Tunerlabs File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
33 changes: 33 additions & 0 deletions
33
src/database/migrations/20240221051434-add-mentee-visibility-policies.js
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,33 @@ | ||
'use strict' | ||
|
||
/** @type {import('sequelize-cli').Migration} */ | ||
module.exports = { | ||
async up(queryInterface, Sequelize) { | ||
await queryInterface.addColumn('organization_extension', 'mentee_visibility_policy', { | ||
type: Sequelize.STRING, | ||
defaultValue: 'CURRENT', | ||
}) | ||
|
||
await queryInterface.addColumn('organization_extension', 'external_mentee_visibility_policy', { | ||
type: Sequelize.STRING, | ||
defaultValue: 'CURRENT', | ||
}) | ||
|
||
await queryInterface.addColumn('user_extensions', 'external_mentee_visibility', { | ||
type: Sequelize.STRING, | ||
defaultValue: 'CURRENT', | ||
}) | ||
|
||
await queryInterface.addColumn('mentor_extensions', 'external_mentee_visibility', { | ||
type: Sequelize.STRING, | ||
defaultValue: 'CURRENT', | ||
}) | ||
}, | ||
|
||
async down(queryInterface, Sequelize) { | ||
await queryInterface.removeColumn('organization_extension', 'mentee_visibility_policy') | ||
await queryInterface.removeColumn('organization_extension', 'external_mentee_visibility_policy') | ||
await queryInterface.removeColumn('user_extensions', 'external_mentee_visibility') | ||
await queryInterface.removeColumn('mentor_extensions', 'external_mentee_visibility') | ||
}, | ||
} |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
This should not come in your PR.
Please fetch the latest pull and correct this