NOREF Remove unused deprecated search queries #712
Merged
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
StevenWadeOddball
changed the title
# Conflicts: # MINT.postman_collection.json
StevenWadeOddball
requested review from
OddTomBrooks and
garyjzhao
and removed request for
a team
|
This ticket looks more like its BE related. Not 100% sure why the FE/me was tagged 😂 . |
Thanks @garyjzhao Gary! Yeah, you are right. I'm pretty sure it's because of the graph global type being modified (because of the schema changing, even though it doesn't actually impact any implemented queries). Feel free to disregard! |
OddTomBrooks
approved these changes
Sep 26, 2023
StevenWadeOddball
deleted the
NOREF/remove_deprecated_change_search_resolvers
branch
ClayBenson94
added a commit
that referenced
this pull request
Sep 28, 2023
* Change from Question verbiage to Discussion verbiage per Figma * deleted format activity * update recent activity in home page table * only show the discussions if there is more than 0 * various empty states * add a period after the link * working through the styles * Starting a discussion label updates * adding various text updates * add more copy eidts * answer -> reply * good work so far * add a replies compnent * resplies * show/hide replies * wip on replies * specificity over BANGimportant * DRYing code by having DiscussionUserInfo * pass down index * cleaning up padding/margins * wip * pass down replies object as prop * add getDaysElapsed function * implement the date adn time to show on replies * fix typeerror and more conditional rendering * remove comment * remove hasEditAccess * drying up some code * adding the connected styles * Add the conected border styles * Build(deps): bump github.com/contribsys/faktory from 1.7.0 to 1.8.0 (#711) Bumps [github.com/contribsys/faktory](https://github.com/contribsys/faktory) from 1.7.0 to 1.8.0. - [Release notes](https://github.com/contribsys/faktory/releases) - [Changelog](https://github.com/contribsys/faktory/blob/main/Changes.md) - [Commits](contribsys/faktory@v1.7.0...v1.8.0) --- updated-dependencies: - dependency-name: github.com/contribsys/faktory dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * desctructure the prop to only use replies * Read more toggle when there are more than four replies * change div to span so no div within p descendant error * major progress * increase the character limit * EASI-3236: skip_tests feature for manual deployment (#709) EASI-3236: init skip_tests feature for manual deployment * [EASi 3154] links as documents be (#710) * feat: add the ability to add a document which a link to an outside document * feat: add unit test for a linked document * feat: add note about the fact that we dont virus scan a linked document. Add URL format validation and unit test * feat: add postman query for upoloading a linked document * feat: constraint to not allow url to be null if is_link, and require null if !is_link. More explicit url validation * Added tests for other types of URLs --------- Co-authored-by: ClayBenson94 <[email protected]> * adjust the spacing between the replies * made some more chagnes * [EASI-2842] - Model home table updates (#705) * Modified columns on home tables and updated MAC heading text * Update snap * Merged all model plan tables * Fixed model link and adjusted header table styling * Fix mispelled basics and change parent category map key --------- Co-authored-by: Clay Benson <[email protected]> * [NOREF] Split docker compose files (#707) * Split docker compose files by frontend + backend, rather than by regular/override. Remove most duplicative env vars * Fix typo in docker compose file * Use PGPASSWORD instead of PGPASS * Use PGPASS and PGPASSWORD * Remove unused Dockerfiles (client CI and cypress). Rename Dockerfile.client to Dockerfile.frontend, refactor to no longer copy source and use alpine base image * Remove templates for emails (they are embedded in the binary). Refactor dockerfile for backend. * [NOREF] Fix dockerfile copy command copying to wrong directory (#714) Fix dockerfile copy command copying to wrong directory * Build(deps): bump docker/setup-buildx-action from 2 to 3 (#715) Bumps [docker/setup-buildx-action](https://github.com/docker/setup-buildx-action) from 2 to 3. - [Release notes](https://github.com/docker/setup-buildx-action/releases) - [Commits](docker/setup-buildx-action@v2...v3) --- updated-dependencies: - dependency-name: docker/setup-buildx-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [NOREF] Delete RDS manual snapshots (#718) Delete RDS manual snapshots until there are 10 remaining to ensure we do not hit the limit of 100 manual snapshots * [Easi 3130]/discussion tagging adr (#706) * feat: discussion tagging ADR Co-authored-by: Patrick Segura <[email protected]> * NOREF Remove unused deprecated search queries (#712) * chore: remove unused deprecated search queries. Base resolver code retained * feat: remove unused queries from postman collection * redo how the accordion works * move accordion to its own component * add stautus pback * Remove commented code * remove more commented code * update snaps for ton of things * update tests * fixing duplicated name migrations * Build(deps): bump the other group with 2 updates (#716) Bumps the other group with 2 updates: [github.com/99designs/gqlgen](https://github.com/99designs/gqlgen) and [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go). Updates `github.com/99designs/gqlgen` from 0.17.37 to 0.17.38 - [Release notes](https://github.com/99designs/gqlgen/releases) - [Changelog](https://github.com/99designs/gqlgen/blob/master/CHANGELOG.md) - [Commits](99designs/gqlgen@v0.17.37...v0.17.38) Updates `github.com/aws/aws-sdk-go` from 1.45.12 to 1.45.15 - [Release notes](https://github.com/aws/aws-sdk-go/releases) - [Commits](aws/aws-sdk-go@v1.45.12...v1.45.15) --- updated-dependencies: - dependency-name: github.com/99designs/gqlgen dependency-type: direct:production update-type: version-update:semver-patch dependency-group: other - dependency-name: github.com/aws/aws-sdk-go dependency-type: direct:production update-type: version-update:semver-patch dependency-group: other ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Clay Benson <[email protected]> * linting * Update discussion e2e test * fix: updated plan discussion unit tests to drop status * using regex to find the correct button --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Justin Woodson <[email protected]> Co-authored-by: Steven Wade <[email protected]> Co-authored-by: ClayBenson94 <[email protected]> Co-authored-by: Patrick Segura <[email protected]> Co-authored-by: Patrick Segura <[email protected]> Co-authored-by: Tom Brooks <[email protected]>
OddTomBrooks
added a commit
that referenced
this pull request
Sep 29, 2023
* feat: removed resolutions from discussion replies for threaded discussions * feat: removed discussion status from various front-end files * fix: updated function call in broken test * feat: removed discussion status from go and sql * chore: removed references to discussion status from plan discussion sql * [EASI-3129] FE -- threaded replies (#719) * Change from Question verbiage to Discussion verbiage per Figma * deleted format activity * update recent activity in home page table * only show the discussions if there is more than 0 * various empty states * add a period after the link * working through the styles * Starting a discussion label updates * adding various text updates * add more copy eidts * answer -> reply * good work so far * add a replies compnent * resplies * show/hide replies * wip on replies * specificity over BANGimportant * DRYing code by having DiscussionUserInfo * pass down index * cleaning up padding/margins * wip * pass down replies object as prop * add getDaysElapsed function * implement the date adn time to show on replies * fix typeerror and more conditional rendering * remove comment * remove hasEditAccess * drying up some code * adding the connected styles * Add the conected border styles * Build(deps): bump github.com/contribsys/faktory from 1.7.0 to 1.8.0 (#711) Bumps [github.com/contribsys/faktory](https://github.com/contribsys/faktory) from 1.7.0 to 1.8.0. - [Release notes](https://github.com/contribsys/faktory/releases) - [Changelog](https://github.com/contribsys/faktory/blob/main/Changes.md) - [Commits](contribsys/faktory@v1.7.0...v1.8.0) --- updated-dependencies: - dependency-name: github.com/contribsys/faktory dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * desctructure the prop to only use replies * Read more toggle when there are more than four replies * change div to span so no div within p descendant error * major progress * increase the character limit * EASI-3236: skip_tests feature for manual deployment (#709) EASI-3236: init skip_tests feature for manual deployment * [EASi 3154] links as documents be (#710) * feat: add the ability to add a document which a link to an outside document * feat: add unit test for a linked document * feat: add note about the fact that we dont virus scan a linked document. Add URL format validation and unit test * feat: add postman query for upoloading a linked document * feat: constraint to not allow url to be null if is_link, and require null if !is_link. More explicit url validation * Added tests for other types of URLs --------- Co-authored-by: ClayBenson94 <[email protected]> * adjust the spacing between the replies * made some more chagnes * [EASI-2842] - Model home table updates (#705) * Modified columns on home tables and updated MAC heading text * Update snap * Merged all model plan tables * Fixed model link and adjusted header table styling * Fix mispelled basics and change parent category map key --------- Co-authored-by: Clay Benson <[email protected]> * [NOREF] Split docker compose files (#707) * Split docker compose files by frontend + backend, rather than by regular/override. Remove most duplicative env vars * Fix typo in docker compose file * Use PGPASSWORD instead of PGPASS * Use PGPASS and PGPASSWORD * Remove unused Dockerfiles (client CI and cypress). Rename Dockerfile.client to Dockerfile.frontend, refactor to no longer copy source and use alpine base image * Remove templates for emails (they are embedded in the binary). Refactor dockerfile for backend. * [NOREF] Fix dockerfile copy command copying to wrong directory (#714) Fix dockerfile copy command copying to wrong directory * Build(deps): bump docker/setup-buildx-action from 2 to 3 (#715) Bumps [docker/setup-buildx-action](https://github.com/docker/setup-buildx-action) from 2 to 3. - [Release notes](https://github.com/docker/setup-buildx-action/releases) - [Commits](docker/setup-buildx-action@v2...v3) --- updated-dependencies: - dependency-name: docker/setup-buildx-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [NOREF] Delete RDS manual snapshots (#718) Delete RDS manual snapshots until there are 10 remaining to ensure we do not hit the limit of 100 manual snapshots * [Easi 3130]/discussion tagging adr (#706) * feat: discussion tagging ADR Co-authored-by: Patrick Segura <[email protected]> * NOREF Remove unused deprecated search queries (#712) * chore: remove unused deprecated search queries. Base resolver code retained * feat: remove unused queries from postman collection * redo how the accordion works * move accordion to its own component * add stautus pback * Remove commented code * remove more commented code * update snaps for ton of things * update tests * fixing duplicated name migrations * Build(deps): bump the other group with 2 updates (#716) Bumps the other group with 2 updates: [github.com/99designs/gqlgen](https://github.com/99designs/gqlgen) and [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go). Updates `github.com/99designs/gqlgen` from 0.17.37 to 0.17.38 - [Release notes](https://github.com/99designs/gqlgen/releases) - [Changelog](https://github.com/99designs/gqlgen/blob/master/CHANGELOG.md) - [Commits](99designs/gqlgen@v0.17.37...v0.17.38) Updates `github.com/aws/aws-sdk-go` from 1.45.12 to 1.45.15 - [Release notes](https://github.com/aws/aws-sdk-go/releases) - [Commits](aws/aws-sdk-go@v1.45.12...v1.45.15) --- updated-dependencies: - dependency-name: github.com/99designs/gqlgen dependency-type: direct:production update-type: version-update:semver-patch dependency-group: other - dependency-name: github.com/aws/aws-sdk-go dependency-type: direct:production update-type: version-update:semver-patch dependency-group: other ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Clay Benson <[email protected]> * linting * Update discussion e2e test * fix: updated plan discussion unit tests to drop status * using regex to find the correct button --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Justin Woodson <[email protected]> Co-authored-by: Steven Wade <[email protected]> Co-authored-by: ClayBenson94 <[email protected]> Co-authored-by: Patrick Segura <[email protected]> Co-authored-by: Patrick Segura <[email protected]> Co-authored-by: Tom Brooks <[email protected]> * Fix conflicts in postman collection * chore: updated postman collection --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Clay Benson <[email protected]> Co-authored-by: Gary Zhao <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Justin Woodson <[email protected]> Co-authored-by: Steven Wade <[email protected]> Co-authored-by: Patrick Segura <[email protected]> Co-authored-by: Patrick Segura <[email protected]>
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.
Changes and Description
How to test this change
PR Author Review Checklist
PR Reviewer Guidelines