-
Notifications
You must be signed in to change notification settings - Fork 35
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
Migrate submissions to new API #3010
Merged
cesarvarela
merged 11 commits into
responsible-ai-collaborative:staging
from
clari182:fix/submissions-new-api
Aug 15, 2024
Merged
Migrate submissions to new API #3010
cesarvarela
merged 11 commits into
responsible-ai-collaborative:staging
from
clari182:fix/submissions-new-api
Aug 15, 2024
Conversation
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
Collaborator
clari182
commented
Aug 5, 2024
•
edited
Loading
edited
- closes Migrate submissions to new API #3009
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## staging #3010 +/- ##
===========================================
- Coverage 96.79% 93.21% -3.59%
===========================================
Files 30 96 +66
Lines 8680 10297 +1617
Branches 172 656 +484
===========================================
+ Hits 8402 9598 +1196
- Misses 274 659 +385
- Partials 4 40 +36 ☔ View full report in Codecov by Sentry. |
cesarvarela
approved these changes
Aug 15, 2024
cesarvarela
merged commit Aug 15, 2024
a21e87b
into
responsible-ai-collaborative:staging
13 of 20 checks passed
cesarvarela
added a commit
that referenced
this pull request
Aug 19, 2024
* Reduce space usage, limit number of fields shown * wip * wip * Color-code tables * Create confirmemail.spec.ts * Delete confirmemail.cy.js * Use test from utils * Make gray default * Fix variants table query * Migrate fields to local API (#2801) * Forward graphql requests to netlify function * Trigger Build * update lockfile * Avoid having to introspect schema for every request * Allow anonymous read only queries * Allow anonymous read only queries * Trigger Build * Add quickadds resolver * Split resolvers into separate files * Separate auth in its own file * Add jest and tests * Add script to boot local api server for development purposes * Add quickadd insert field * Add github workflows * Trigger build remove Trigger build Trigger build fsdfsf Trigger build Trigger build bigint patch use require * Update codegen tool config * Reuse mongodb client from context * Update codege * Strongly typed environment config * Increase function timeout * Update bson version * Make environment variables available to gatsby functions * Fix ts error * Trigger build * Make errors more verbose * Add missing variables * Disable env validation * Use graphql-http to prevent event emitter leaks * Enable env validation * New bson version exports ObjectId instead of ObjectID * Update lockfile * Trigger build * Remove unused query * bue * Add support for filters, sorting and pagination * Remove unused package * Remove unused file * allow api tests to be cancelled * Delete old test file * Run api tests when pushing staging * update test utils * Trigger build * Force exit to let tests exit * remove old api endpoint * Update client side queries * Udpate docs * Use vars for api and group ids * convert to import jic * Trigger build * Trigger build * Update workflows * Trigger build * Add missing variables * Fix error when object id is already serialized * Update workflow file * Improve error message * update readme * Remove unused vars * Remove unused packages * Log errors to rollbar * Trigger build * Add rollbar token variable * Trigger build * fix variable name * Trigger build * Add function to programmatically generate query fields * Add programmatic mutation fields function * Add tests for update methods * update client side queries * Add field queries tests * add tests for insert and update mutations * add delete mutations programmatic tests * Configure fixtures * Tests suite needs to be run in a single process * And missing dynamic fields and improve fixture design * Add roles expectations * delete redundant tests * Add incidents and report queries * Add report translations resolver * add missing dotenv config * Update tests to mock realm api * remove unrelated files * Remove unrelated changes * Remove unrelated changes * Remove access to test user credentials * Remove unused * Use dummy values on api test workflow * Fix wrong database being mocked * Fix schema issue * cleanup * Add incidents and reports field implementation * Implement linkReportsToIncidents mutation * Typed return * Delete old implementation * Add incidents/edit.spec * Add incidents/history.spec * Add incidents/new.spec * Delete incidents/new.cy incidents/history.cy and incidents/edit.cy * Add reports test api fixtures and improve fixtures expressiveness * Migrate entities collection * Add incidents fixture and update fixture structure * Add users fixture * Add utility functions to resolve and configure relationships * Make sure inputs are required * Add special type for available fields * Update test fixtures and fix permissions * add flag report mutation and tests * Add date modified fields * Fix upsertOne issues * Remove leftover * Add test to report to incident linking mutation * Add submissions collection to api * Add full e2e workflow * Trigger Build * update lockfile * Fix history latest version test * Fix new spec errors * Add in memory mongo server * remove comment * Wait for memory server to start * Trigger Build * Remove quotes * Trigger Build * Fix typo in name * Get cite tests to work fully e2e * Rename intercept for consistency * Add missing waitForRequest * get submission form working with new api * Add missing env variable * Trigger Build * Add missing variable * Trigger Build * Update users table components * Add similarity flag mutation * Add update one report mutation * Get incident report edit form to work * update lockfile * Get dinamic citation page to work * Get incident handling to work * Update tests * Update entity type to use date objects * Fix schema stiching warnings * Fix/silence warnings * Rename to prevent artifact collision * Update history related tests * Trigger Build * Add docs to server utils * Silence warning * Run mocked tests separately. * Trigger Build * Make sure to give time to autocomplete * Fix tests * Add createVariant mutation and separate types into files to deal with circular dep * Delete custom mutation from remote * Update variants related components and tests * Trigger Build * Add memory mongo command * Update tests * avoid report artifact collision * Trigger Build * update test * Fix incidents app * Remove debug * Separate incident data type into its own file --------- Co-authored-by: Clara Youdale <clara@botsfactory.io> * Add full e2e workflows to production and staging * Add classifications collection to api * delete unnecessary merge configs * Migrate taxa collection to api * Revert "Add classifications collection to api" This reverts commit 0da2bc0. * setNavCollapsed for quick access * Change query to filter on Variants table * Update workflows * Remove the outdated `variants.cy.js` and use `playwright/e2e-full/apps/variants.spec.ts` instead. * Enable static type checking of Apollo client calls (#3012) * Configure graphql codegen to work with Apollo client and enable static checking * Update queries to use new codegen function * Fix anonymous mutation * Add updated codegen files * Add comment * Add codegen babel plugin to remove generated query map from production bundle * Add generated fragment maskings * update lockfile * remove unnecessary mock * style * Delete migrated files (#3003) * Delete migrated files * Delete function file * Delete report translations resolver * Change FIND_INCIDENTS query filter to use the new approach * Migrate submissions to new API (#3010) * Migrate submissions to new API * Remove unused adminData * Fix updateSubmission mutation with new API changes * Update SubmissionEditModal.js * Add .ts to eslintrc * Check for incident_ids if null * Add new user query to not fetch adminData * Add generated files for new query * Revert to previous user roles query * Silence schema merge warning (#3024) * Fix staging issues (#3026) * Add fixes to relationship handling from #1072 * Return link validation errors instead of throwing so they are shown in graphql response * Update outdated query variables * Make sure query paramter is int * Add tests --------- Co-authored-by: Luna McNulty <l.mcnulty@riseup.net> Co-authored-by: Clara Youdale <clara@botsfactory.io> Co-authored-by: Pablo Costa <pablo@botsfactory.io> Co-authored-by: Kevin <kepae@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.