-
Notifications
You must be signed in to change notification settings - Fork 55
[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
[PROD RELEASE] - Q2 #823
Changes from 1 commit
6163394
0372f83
06280ae
561906e
5ac2a16
99a5053
0b40100
fef6473
0afeeac
5907e42
4adf1e9
4b7990b
c608346
cffc8c0
bf83bcb
dc39219
22cc474
472acdf
5a0b44f
7c63711
f0d083f
aa60669
47664c8
1978a22
3cac325
9ba39a3
7ad57ec
8dfcdf0
4a13b50
350d155
c922d0c
c3e8fca
5329b81
ca59b33
dfa1ca3
f0e40c1
a0e0d8b
7be3b14
1bd520d
7dcf79a
93c1ca9
b6f6b8b
45f8ff2
21edae9
840f155
d093fe5
8411731
43dc20c
0a5d052
c42c1d4
7adb0cc
51c50d8
aeb7b48
af846ab
709601f
4543e5e
9f7dc48
7f40cdb
8e2881d
f7d0990
4753022
f852b5a
2ee4f9b
2fd1652
d5296d3
4350c1f
3a64f1d
cfbbd56
c36ca64
f181533
0f4db68
c7d8d37
d2aa3e9
69eb97a
675858b
613defe
e3303fe
d83097b
68fdf7b
7383c70
aaea12c
35d27e0
1102fdc
fe085cf
42485c5
944692c
2d20d16
3ec8ebd
364e74b
d691100
3a4fc53
6224d47
d6bd47e
07973aa
c43ace7
9ba2c67
4b1e2b3
4defcf7
cc46a74
3bf61b2
e19a9d1
d4d3790
3a8a1fc
ea2ac33
5151cfa
d31503e
8507660
09249e6
d10aa02
4f2ea1a
5caaf0d
cf0236d
d9d9cd4
aa20437
da294ad
bbb0d6b
dea4e18
759a4db
160c4d8
db388bc
ded8a7c
7c29993
cc92278
dc019e3
79fdf6f
4b8c9f7
0c54ecf
223b7bd
82dd797
9793dcc
5b29f65
388e9f5
a282f0b
ff6f62a
0a3317a
aa5cc62
f7579a0
2065478
724d028
21b079b
77b0fb0
a29f1c9
deaed3a
d4ed7a9
91f23ab
b7f5bf5
4c45e27
595e046
fb9b8a9
31a2b66
ae5952d
c16d06e
190c04c
4c98fe7
0ad22c3
cc5a28d
b52b16e
5fbd197
49356d6
e34f785
6f4cb68
024b70f
1a5016f
24d2fba
a26bd8f
37bd5ed
339c8ab
9be8c5f
d95af0d
126ea0b
3875c3e
92ee79e
70e8e40
7bae3fd
12fab3a
71f598c
f8a6786
596d9b1
9d70104
2eb2d21
f6138fe
8612e02
81effa6
25b4d80
6080e8e
6557460
8e30d1b
8d2840c
70422b7
c3390fc
66a6ffa
a09a791
852c89a
69fe32b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
- Loading branch information
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -13,7 +13,7 @@ module.exports = freq => new Promise((resolve, reject) => { | |
console.log("start permission check"); | ||
const opportunityId = _.parseInt(freq.params.id); | ||
const currentUserId = freq.authUser.userId; | ||
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. Ensure that |
||
return models.CopilotOpportunity.find({ | ||
return models.CopilotOpportunity.findOne({ | ||
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 adding error handling for the |
||
where: { | ||
id: opportunityId, | ||
}, | ||
|
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.
Consider adding validation to ensure
freq.params.id
is a valid number before using_.parseInt
. This could prevent potential issues if the parameter is not a number.