-
Notifications
You must be signed in to change notification settings - Fork 56
[PROD RELEASE] - Q2 #823
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
Merged
[PROD RELEASE] - Q2 #823
Changes from 3 commits
Commits
Show all changes
194 commits
Select commit
Hold shift + click to select a range
6163394
feat: apply for copilot opportunity
hentrymartin 0372f83
deploy pr branch to develo[
hentrymartin 06280ae
add migration
561906e
deploy to dev
5ac2a16
Fix migration script
99a5053
fix: permission issue to delete copilot invite
hentrymartin 0b40100
fix: permission issue to delete copilot invite
hentrymartin fef6473
fix: removed circle ci config
hentrymartin 0afeeac
Merge pull request #800 from topcoder-platform/pm-1077
hentrymartin 5907e42
Make copilot opportunity details route public
4adf1e9
removed circle config
hentrymartin 4b7990b
Merge pull request #799 from topcoder-platform/PM-589-add-migration
hentrymartin c608346
Merge branch 'develop' into PM-577
hentrymartin cffc8c0
updated swagger doc
hentrymartin bf83bcb
fix: removed validation
hentrymartin dc39219
fix: reference error
hentrymartin 22cc474
fix: model
hentrymartin 472acdf
fix: model
hentrymartin 5a0b44f
fix: model
hentrymartin 7c63711
fix: model
hentrymartin f0d083f
fix: model
hentrymartin aa60669
fix: model
hentrymartin 47664c8
fix: model
hentrymartin 1978a22
fix: validations
hentrymartin 3cac325
removed unnecessary file
hentrymartin 9ba39a3
enable AI Review buddy
kkartunov 7ad57ec
fix: added notes column
hentrymartin 8dfcdf0
fix: added notes from request
hentrymartin 4a13b50
updated swagger file
hentrymartin 350d155
debug logs
hentrymartin c922d0c
added notes to model
hentrymartin c3e8fca
sanitize notes
hentrymartin 5329b81
sanitize notes
hentrymartin ca59b33
removed console log
hentrymartin dfa1ca3
Merge pull request #798 from topcoder-platform/PM-577
hentrymartin f0e40c1
PM-1068 refactor: allow PMs to view all copilot requests
a0e0d8b
fix: return existing application instead of throwing an error
hentrymartin 7be3b14
fix: return existing application instead of throwing an error
hentrymartin 1bd520d
removed circle branch config
hentrymartin 7dcf79a
Merge pull request #802 from topcoder-platform/pm-577_1
hentrymartin 93c1ca9
feat: list copilot applications
hentrymartin b6f6b8b
added circle config
hentrymartin 45f8ff2
changed route method
hentrymartin 21edae9
updated permission method
hentrymartin 840f155
updated permission method
hentrymartin d093fe5
updated permission method
hentrymartin 8411731
added opp id to the where clause
hentrymartin 43dc20c
show all applications for pm
hentrymartin 0a5d052
Merge pull request #801 from topcoder-platform/PM-1068
himaniraghav3 c42c1d4
fix: allow only copilots to apply for opportunity
hentrymartin 7adb0cc
removed dev deployment circle config
hentrymartin 51c50d8
Merge pull request #803 from topcoder-platform/pm-855
hentrymartin aeb7b48
feat: assign copilot opportunity
hentrymartin af846ab
deploy feature branch to dev
hentrymartin 709601f
deploy feature branch to dev
hentrymartin 4543e5e
fix: return object
hentrymartin 9f7dc48
fix: return object
hentrymartin 7f40cdb
fix: return object
hentrymartin 8e2881d
updated swagger docs
hentrymartin f7d0990
fix: make notes optional
hentrymartin 4753022
removed circle config
hentrymartin f852b5a
Merge pull request #806 from topcoder-platform/pm-578_1
hentrymartin 2ee4f9b
fix: lint
hentrymartin 2fd1652
fix: check if the user is already a copilot
hentrymartin d5296d3
fix: allow project managers to view applications of opportunity not b…
hentrymartin 4350c1f
fix: allow project managers to view applications of opportunity not b…
hentrymartin 3a64f1d
fix: allow project managers to view applications of opportunity not b…
hentrymartin cfbbd56
removed not needed circle config
hentrymartin c36ca64
Merge pull request #807 from topcoder-platform/pm-1067_1
hentrymartin f181533
Merge branch 'develop' into pm-1168
hentrymartin 0f4db68
Merge pull request #805 from topcoder-platform/pm-1168
kkartunov c7d8d37
PM-1207 Make copilot oppportunity details api public
d2aa3e9
Fix accidentally removed code
69eb97a
fix: invite user when assigning as copilot
hentrymartin 675858b
fix: invite user when assigning as copilot
hentrymartin 613defe
fix: invite user when assigning as copilot
hentrymartin e3303fe
fix: invite user when assigning as copilot
hentrymartin d83097b
fix: invite user when assigning as copilot
hentrymartin 68fdf7b
fix: invite user when assigning as copilot
hentrymartin 7383c70
fix: invite user when assigning as copilot
hentrymartin aaea12c
fix: invite user when assigning as copilot
hentrymartin 35d27e0
fix: removed circle config
hentrymartin 1102fdc
fix: added migration to set application id in project member invite
hentrymartin fe085cf
fix: added migration to set application id in project member invite
hentrymartin 42485c5
fix: complete the copilot application on accepting invite
hentrymartin 944692c
Merge pull request #808 from topcoder-platform/PM-1207
kkartunov 2d20d16
fix: complete the copilot application on accepting invite
hentrymartin 3ec8ebd
fix: complete the copilot application on accepting invite
hentrymartin 364e74b
fix: complete the copilot application on accepting invite
hentrymartin d691100
fix: complete the copilot application on accepting invite
hentrymartin 3a4fc53
fix: added new application status
hentrymartin 6224d47
Merge pull request #809 from topcoder-platform/pm-1168_1
hentrymartin d6bd47e
fix: QA feedbacks for assigning copilots
hentrymartin 07973aa
fix: move application to pending when all invites are rejected
hentrymartin c43ace7
fix: move application to pending when all invites are rejected
hentrymartin 9ba2c67
fix: move application to pending when all invites are rejected
hentrymartin 4b1e2b3
fix: move application to pending when all invites are rejected
hentrymartin 4defcf7
fix: move application to pending when all invites are rejected
hentrymartin cc46a74
fix: move application to pending when all invites are rejected
hentrymartin 3bf61b2
fix: move application to pending when all invites are rejected
hentrymartin e19a9d1
fix: move application to pending when all invites are rejected
hentrymartin d4d3790
debug
hentrymartin 3a8a1fc
debug
hentrymartin ea2ac33
debug
hentrymartin 5151cfa
Merge pull request #810 from topcoder-platform/pm-1168_2
kkartunov d31503e
fix: don't show applied on for completed project
hentrymartin 8507660
fix: don't show applied on for completed project
hentrymartin 09249e6
fix: don't show applied on for completed project
hentrymartin d10aa02
fix: don't show applied on for completed project
hentrymartin 4f2ea1a
fix: don't show applied on for completed project
hentrymartin 5caaf0d
fix: don't show applied on for completed project
hentrymartin cf0236d
removed circle config
hentrymartin d9d9cd4
Merge pull request #811 from topcoder-platform/pm-1168_2
hentrymartin aa20437
DB schema config parameter
jmgasper da294ad
Merge pull request #812 from topcoder-platform/db_schema
jmgasper bbb0d6b
Add schema to umzug config as well
jmgasper dea4e18
Back out circle ci test
jmgasper 759a4db
Merge pull request #813 from topcoder-platform/db_schema
jmgasper 160c4d8
fix: send source as part of project member invite
hentrymartin db388bc
fix: send source as part of project member invite
hentrymartin ded8a7c
fix: send source as part of project member invite
hentrymartin 7c29993
fix: send source as part of project member invite
hentrymartin cc92278
fix: update copilot request based on the source of invite
hentrymartin dc019e3
fix: send source to accept or decline invite API
hentrymartin 79fdf6f
feat: get roles by role name
hentrymartin 4b8c9f7
deploy feature branch
hentrymartin 0c54ecf
deploy feature branch
hentrymartin 223b7bd
deploy feature branch
hentrymartin 82dd797
deploy feature branch
hentrymartin 9793dcc
fix: get role by role name
hentrymartin 5b29f65
modified timeout
hentrymartin 388e9f5
Merge pull request #814 from topcoder-platform/pm-1169
kkartunov a282f0b
fix: cancel all copilot requests for a project when manually copilot …
hentrymartin ff6f62a
fix: cancel all copilot requests for a project when manually copilot …
hentrymartin 0a3317a
fix: cancel all copilot requests for a project when manually copilot …
hentrymartin aa5cc62
fix: cancel all copilot requests for a project when manually copilot …
hentrymartin f7579a0
fix: cancel all copilot requests for a project when manually copilot …
hentrymartin 2065478
fix: cancel all copilot requests for a project when manually copilot …
hentrymartin 724d028
fix: cancel all copilot requests for a project when manually copilot …
hentrymartin 21b079b
fix: cancel all copilot requests for a project when manually copilot …
hentrymartin 77b0fb0
Merge pull request #816 from topcoder-platform/pm-1169_1
kkartunov a29f1c9
rebased from develop branch
hentrymartin deaed3a
rebased from develop branch
hentrymartin d4ed7a9
rebased from develop branch
hentrymartin 91f23ab
rebased from develop branch
hentrymartin b7f5bf5
fix: sync issue between db and es
hentrymartin 4c45e27
fix: sync issue
hentrymartin 595e046
Merge pull request #817 from topcoder-platform/pm-1169_2
hentrymartin fb9b8a9
Merge branch 'develop' into pm-1173
hentrymartin 31a2b66
fix: sync issue
hentrymartin ae5952d
fix: added external action email kaufka type
hentrymartin c16d06e
fix: added external action email kaufka type
hentrymartin 190c04c
fix: added external action email kaufka type
hentrymartin 4c98fe7
fix: added external action email kaufka type
hentrymartin 0ad22c3
fix: added external action email kaufka type
hentrymartin cc5a28d
fix: send emails to PMs and creator
hentrymartin b52b16e
fix: send emails to PMs and creator
hentrymartin 5fbd197
fix: send emails to PMs and creator
hentrymartin 49356d6
fix: review comments
hentrymartin e34f785
fix: review comments
hentrymartin 6f4cb68
Merge pull request #815 from topcoder-platform/pm-1173
hentrymartin 024b70f
fix: added work manager url
hentrymartin 1a5016f
fix: retain query param
hentrymartin 24d2fba
fix: removed validation for source as default is work manager
hentrymartin a26bd8f
fix: removed validation for source as default is work manager
hentrymartin 37bd5ed
Merge pull request #818 from topcoder-platform/pm-1173_1
hentrymartin 339c8ab
debug apply copilot
hentrymartin 9be8c5f
debug apply copilot
hentrymartin d95af0d
fix: use correct field
hentrymartin 126ea0b
fix: use correct field
hentrymartin 3875c3e
fix: lowercase null exception
hentrymartin 92ee79e
removed debug logs
hentrymartin 70e8e40
removed circle ci configs
hentrymartin 7bae3fd
Merge pull request #820 from topcoder-platform/pm-1175_2
hentrymartin 12fab3a
fix: send project member invite email via external email event
hentrymartin 71f598c
Merge pull request #821 from topcoder-platform/pm-1356
kkartunov f8a6786
fix: send members as part of opportunity
hentrymartin 596d9b1
fix: send members as part of opportunity
hentrymartin 9d70104
fix: send members as part of opportunity
hentrymartin 2eb2d21
fix: send members as part of opportunity
hentrymartin f6138fe
fix: send members as part of opportunity
hentrymartin 8612e02
fix: send members as part of opportunity
hentrymartin 81effa6
fix: send members as part of opportunity
hentrymartin 25b4d80
debug auth user
hentrymartin 6080e8e
debug auth user
hentrymartin 6557460
debug auth user
hentrymartin 8e30d1b
Merge pull request #824 from topcoder-platform/pm-1273
kkartunov 8d2840c
feat: cancel copilot opportunity
hentrymartin 70422b7
fix: undefined
hentrymartin c3390fc
fix: lint
hentrymartin 66a6ffa
fix: cancel endpoint
hentrymartin a09a791
fix: cancel endpoint
hentrymartin 852c89a
fix: cancel endpoint
hentrymartin 69fe32b
Merge pull request #825 from topcoder-platform/pm-1378
hentrymartin 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 hidden or 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 |
|---|---|---|
|
|
@@ -25,8 +25,8 @@ router.get(`/${apiVersion}/projects/health`, (req, res) => { | |
|
|
||
| // List of public routes | ||
| const publicRoutes = [ | ||
| `/${apiVersion}/projects/copilots/opportunities`, | ||
| `/${apiVersion}/projects/copilot/opportunity/:id(\\d+)`, | ||
| new RegExp(`^/${apiVersion}/projects/copilots/opportunities$`), | ||
| new RegExp(`^/${apiVersion}/projects/copilot/opportunity/\\d+$`), | ||
| ]; | ||
|
|
||
| // All project service endpoints need authentication | ||
|
|
@@ -35,7 +35,7 @@ const jwtAuth = require('tc-core-library-js').middleware.jwtAuthenticator; | |
| router.all( | ||
| RegExp(`\\/${apiVersion}\\/(copilots|projects|timelines|orgConfig|customer-payments)(?!\\/health).*`), | ||
| (req, res, next) => { | ||
| if (publicRoutes.some(route => req.path.match(new RegExp(`^${route}$`)))) { | ||
| if (publicRoutes.some(routeRegex => routeRegex.test(req.path))) { | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Consider renaming |
||
| return next(); | ||
| } | ||
| // JWT authentication | ||
|
|
||
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.
The regex pattern for the copilot opportunity endpoint has been changed from
opportunities/:id(\\d+)toopportunity/\\d+. Ensure that this change is intentional and that the endpoint is correctly updated in other parts of the application to reflect this singular form and pattern.