-
Notifications
You must be signed in to change notification settings - Fork 795
Adding validation for collection availability #5185
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
Adding validation for collection availability #5185
Conversation
SarthakJain26
left a comment
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.
LGTM
|
@prateekch33 can you please fix the go import order, it is causing the backend checks to fail |
Signed-off-by: prateekch33 <[email protected]>
Updated email address for Neelanjan Manna. Signed-off-by: Neelanjan Manna <[email protected]> Signed-off-by: prateekch33 <[email protected]>
Signed-off-by: prateekch33 <[email protected]>
Signed-off-by: prateekch33 <[email protected]>
Signed-off-by: prateekch33 <[email protected]>
651d34d to
92e73d7
Compare
Signed-off-by: prateekch33 <[email protected]>
|
@SarthakJain26 I have updated the order of the imports in init.go file, please check |
|
@prateekch33 the build pipeline is still failing, can you please check the import orders. Please let us know if you need any help |
|
@SarthakJain26 I tried fixing the import order in the file, but I don't know why its still failing, let's connect for sometime tomorrow or later, as per your availability to get this fixed. Let me know the medium through which we can connect. |
|
@prateekch33 please try running goimports -w -l ./ in your working terminal. If this does not fixes this, then we will merge this and try to fix in another PR |
Signed-off-by: prateekch33 <[email protected]>
|
@SarthakJain26 I ran that command, and pushed the changes, let's see whether it resolves the issue or not |
Proposed changes
Implemented validation to ensure the specified MongoDB collection exists before proceeding with GraphQL operations.
Types of changes
What types of changes does your code introduce to Litmus? Put an
xin the boxes that applyChecklist
Put an
xin 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. We're here to help! This is simply a reminder of what we are going to look for before merging your code.