Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Release API category (Jul 1) #614

Merged
merged 10 commits into from
Jul 1, 2022
Merged

Release API category (Jul 1) #614

merged 10 commits into from
Jul 1, 2022

Conversation

AaronZyLee
Copy link
Contributor

@AaronZyLee AaronZyLee commented Jun 29, 2022 •
edited
Loading

Description of changes

Feature

  • Added subscription runtime filter support

Fix

  • Fixed implicit timestamps fields not working for search queries bug
  • Feature flag renaming: useFieldNameForPrimaryKeyConnectionField to respectPrimaryKeyAttributesOnConnectionField
  • Added error messaging for boolean type as index field

Ops

  • Removed unused dependencies
  • Updated lint rules

Issue #, if available

Description of how you validated changes

Checklist

  • PR description included
  • yarn test passes
  • Tests are changed or added
  • Relevant documentation is changed or added (and PR referenced)
  • New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@AaronZyLee AaronZyLee requested a review from a team as a code owner June 29, 2022 19:26
@AaronZyLee AaronZyLee marked this pull request as draft June 29, 2022 19:30
@AaronZyLee AaronZyLee marked this pull request as ready for review June 30, 2022 17:56
@AaronZyLee AaronZyLee marked this pull request as draft June 30, 2022 17:56
@AaronZyLee AaronZyLee marked this pull request as ready for review June 30, 2022 19:06
alharris-at
alharris-at previously approved these changes Jun 30, 2022
View reviewed changes
sundersc and others added 10 commits June 30, 2022 15:31
@sundersc @AaronZyLee
test(graphql): fix failing datastore modelgen tests (#575)
3f5a575 
* test(graphql): fix failing datastore modelgen tests

* moved the amplify-app to e2e tests package
@amplify-data-ci @AaronZyLee
chore(release): Publish [ci skip]
2264c96 
 - @aws-amplify/amplify-category-api@3.1.2
 - amplify-category-api-e2e-core@4.0.2
 - amplify-category-api-e2e-tests@3.10.2
 - @aws-amplify/graphql-auth-transformer@0.11.1
 - @aws-amplify/graphql-default-value-transformer@0.5.28
 - @aws-amplify/graphql-function-transformer@0.7.22
 - @aws-amplify/graphql-http-transformer@0.8.22
 - @aws-amplify/graphql-index-transformer@0.12.1
 - @aws-amplify/graphql-maps-to-transformer@1.1.20
 - amplify-category-api-graphql-migration-tests@2.3.2
 - @aws-amplify/graphql-model-transformer@0.14.6
 - @aws-amplify/graphql-predictions-transformer@0.6.22
 - @aws-amplify/graphql-relational-transformer@0.10.1
 - @aws-amplify/graphql-schema-test-library@1.1.1
 - @aws-amplify/graphql-searchable-transformer@0.14.6
 - @aws-amplify/graphql-transformer-core@0.17.6
 - @aws-amplify/graphql-transformer-migrator@1.4.1
 - amplify-category-api-migration-tests@5.0.2
 - amplify-category-api-util-mock@5.1.1
 - graphql-auth-transformer@7.2.39
 - graphql-connection-transformer@5.2.39
 - graphql-dynamodb-transformer@7.2.39
 - graphql-elasticsearch-transformer@5.2.39
 - graphql-function-transformer@3.3.30
 - graphql-http-transformer@5.2.39
 - graphql-key-transformer@3.2.39
 - graphql-predictions-transformer@3.2.39
 - graphql-relational-schema-transformer@2.21.9
 - graphql-transformer-common@4.23.3
 - graphql-transformer-core@7.6.2
 - amplify-category-api-graphql-transformers-e2e-tests@8.1.1
 - graphql-versioned-transformer@5.2.39
@AaronZyLee
fix: timestamp field sort in search queries (#581)
77f3985 
* fix: timestamp field sort in search queries

* address CRs
@lazpavel @AaronZyLee
chore: change flag name for PK properties (#593)
4f4a500
@alharris-at @AaronZyLee
chore: add build target to refresh lockfile, and refreshing lockfile
3e3583a
@alharris-at @AaronZyLee
chore: improve error messaging if an invalid boolean sort key is prov…
c3393d3 
…ided to @Index
@sundersc @AaronZyLee
feat(graphql): add runtime filtering support for subscriptions (#551)
7d6a2e1 
* feat(graphql): subscription runtime filtering

* added dynamic groups support

* update unit and e2e tests

* added more test cases for auth precedence and different operators

* fix enum list type on subscription filter

* update model transformer screenshots

* add test for enum field type

* updated error message on a test
@alharris-at @AaronZyLee
chore: remove unused dependencies based on depcheck report
46c35db
@AaronZyLee @alharris-at
@amplify-data-ci @sundersc @naedx @lazpavel
chore: merge release to main (#616)
31e3c43 
* chore: no-op change to trigger pipelines (#545)

* chore(release): Publish [ci skip]

 - @aws-amplify/amplify-category-api@3.1.1
 - amplify-category-api-dynamodb-simulator@2.4.1
 - amplify-category-api-e2e-core@4.0.1
 - amplify-category-api-e2e-tests@3.10.1
 - @aws-amplify/graphql-auth-transformer@0.11.0
 - @aws-amplify/graphql-default-value-transformer@0.5.27
 - @aws-amplify/graphql-function-transformer@0.7.21
 - @aws-amplify/graphql-http-transformer@0.8.21
 - @aws-amplify/graphql-index-transformer@0.12.0
 - @aws-amplify/graphql-maps-to-transformer@1.1.19
 - amplify-category-api-graphql-migration-tests@2.3.1
 - @aws-amplify/graphql-model-transformer@0.14.5
 - @aws-amplify/graphql-predictions-transformer@0.6.21
 - @aws-amplify/graphql-relational-transformer@0.10.0
 - @aws-amplify/graphql-schema-test-library@1.1.0
 - @aws-amplify/graphql-searchable-transformer@0.14.5
 - @aws-amplify/graphql-transformer-core@0.17.5
 - @aws-amplify/graphql-transformer-interfaces@1.14.4
 - @aws-amplify/graphql-transformer-migrator@1.4.0
 - amplify-category-api-migration-tests@5.0.1
 - amplify-category-api-util-mock@5.1.0
 - graphql-auth-transformer@7.2.38
 - graphql-connection-transformer@5.2.38
 - graphql-dynamodb-transformer@7.2.38
 - graphql-elasticsearch-transformer@5.2.38
 - graphql-function-transformer@3.3.29
 - graphql-http-transformer@5.2.38
 - graphql-key-transformer@3.2.38
 - graphql-predictions-transformer@3.2.38
 - graphql-transformer-core@7.6.1
 - amplify-category-api-graphql-transformers-e2e-tests@8.1.0
 - graphql-versioned-transformer@5.2.38

* API Category Release (#587)

* chore: no-op change to trigger pipelines (#545)

* chore(release): Publish [ci skip]

 - @aws-amplify/amplify-category-api@3.1.1
 - amplify-category-api-dynamodb-simulator@2.4.1
 - amplify-category-api-e2e-core@4.0.1
 - amplify-category-api-e2e-tests@3.10.1
 - @aws-amplify/graphql-auth-transformer@0.11.0
 - @aws-amplify/graphql-default-value-transformer@0.5.27
 - @aws-amplify/graphql-function-transformer@0.7.21
 - @aws-amplify/graphql-http-transformer@0.8.21
 - @aws-amplify/graphql-index-transformer@0.12.0
 - @aws-amplify/graphql-maps-to-transformer@1.1.19
 - amplify-category-api-graphql-migration-tests@2.3.1
 - @aws-amplify/graphql-model-transformer@0.14.5
 - @aws-amplify/graphql-predictions-transformer@0.6.21
 - @aws-amplify/graphql-relational-transformer@0.10.0
 - @aws-amplify/graphql-schema-test-library@1.1.0
 - @aws-amplify/graphql-searchable-transformer@0.14.5
 - @aws-amplify/graphql-transformer-core@0.17.5
 - @aws-amplify/graphql-transformer-interfaces@1.14.4
 - @aws-amplify/graphql-transformer-migrator@1.4.0
 - amplify-category-api-migration-tests@5.0.1
 - amplify-category-api-util-mock@5.1.0
 - graphql-auth-transformer@7.2.38
 - graphql-connection-transformer@5.2.38
 - graphql-dynamodb-transformer@7.2.38
 - graphql-elasticsearch-transformer@5.2.38
 - graphql-function-transformer@3.3.29
 - graphql-http-transformer@5.2.38
 - graphql-key-transformer@3.2.38
 - graphql-predictions-transformer@3.2.38
 - graphql-transformer-core@7.6.1
 - amplify-category-api-graphql-transformers-e2e-tests@8.1.0
 - graphql-versioned-transformer@5.2.38

* Added Windows compatible translation of "hoist-cli" script (#543)

* fix(graphql): handle begin and end vtl keywords in index name (#510)

* fix(graphql): handle begin and end vtl keywords in index name

* updated comments

* chore: cleanup stale s3 buckets, and remove references to amplify-app which we no longer leverage in our test suite (#556)

* feature: add support for default generation of query and gsi names for @Index directive behind a feature flag

* fix(amplify-graphql-auth-transformer, amplify-graphql-relational-transformer, amplify-graphql-transformer-core): support custom primary key with relational directives (#462)

* fix(amplify-graphql-auth-transformer, amplify-graphql-relational-transformer, amplify-graphql-transformer-core): support custom primary key with relational directives

* fix: fixed refactor incorrect hanlding for filter vs connection input names

* fix: merge build errors

* fix: merge caused unit test failures

* fix: merge caused e2e failures

* fix: merge caused e2e failures

* addressing PR feedback

* fix: corrected allowed fields for relational fields with custom PK and restricted auth

* fix: fixed imports and e2e feature flag initialization

* test(graphql): fix failing datastore modelgen tests (#575)

* test(graphql): fix failing datastore modelgen tests

* moved the amplify-app to e2e tests package

Co-authored-by: Al Harris <91494052+alharris-at@users.noreply.github.com>
Co-authored-by: amplify-data-ci <amplify-data-dev+github@amazon.com>
Co-authored-by: naedx <1711099+naedx@users.noreply.github.com>
Co-authored-by: Alexander Harris <alharris@amazon.com>
Co-authored-by: Pavel Lazar <85319655+lazpavel@users.noreply.github.com>

* chore(release): Publish [ci skip]

 - @aws-amplify/amplify-category-api@3.1.2
 - amplify-category-api-e2e-core@4.0.2
 - amplify-category-api-e2e-tests@3.10.2
 - @aws-amplify/graphql-auth-transformer@0.11.1
 - @aws-amplify/graphql-default-value-transformer@0.5.28
 - @aws-amplify/graphql-function-transformer@0.7.22
 - @aws-amplify/graphql-http-transformer@0.8.22
 - @aws-amplify/graphql-index-transformer@0.12.1
 - @aws-amplify/graphql-maps-to-transformer@1.1.20
 - amplify-category-api-graphql-migration-tests@2.3.2
 - @aws-amplify/graphql-model-transformer@0.14.6
 - @aws-amplify/graphql-predictions-transformer@0.6.22
 - @aws-amplify/graphql-relational-transformer@0.10.1
 - @aws-amplify/graphql-schema-test-library@1.1.1
 - @aws-amplify/graphql-searchable-transformer@0.14.6
 - @aws-amplify/graphql-transformer-core@0.17.6
 - @aws-amplify/graphql-transformer-migrator@1.4.1
 - amplify-category-api-migration-tests@5.0.2
 - amplify-category-api-util-mock@5.1.1
 - graphql-auth-transformer@7.2.39
 - graphql-connection-transformer@5.2.39
 - graphql-dynamodb-transformer@7.2.39
 - graphql-elasticsearch-transformer@5.2.39
 - graphql-function-transformer@3.3.30
 - graphql-http-transformer@5.2.39
 - graphql-key-transformer@3.2.39
 - graphql-predictions-transformer@3.2.39
 - graphql-relational-schema-transformer@2.21.9
 - graphql-transformer-common@4.23.3
 - graphql-transformer-core@7.6.2
 - amplify-category-api-graphql-transformers-e2e-tests@8.1.1
 - graphql-versioned-transformer@5.2.39

* fix conflict

* fix conflict

Co-authored-by: Al Harris <91494052+alharris-at@users.noreply.github.com>
Co-authored-by: amplify-data-ci <amplify-data-dev+github@amazon.com>
Co-authored-by: Christopher Sundersingh <83315412+sundersc@users.noreply.github.com>
Co-authored-by: naedx <1711099+naedx@users.noreply.github.com>
Co-authored-by: Alexander Harris <alharris@amazon.com>
Co-authored-by: Pavel Lazar <85319655+lazpavel@users.noreply.github.com>
@marcvberg @AaronZyLee
Update linting rules
310b16b
@AaronZyLee AaronZyLee changed the title Release API category (Jun 29) Release API category (Jul 1) Jul 1, 2022
@AaronZyLee AaronZyLee merged commit 735c19c into release Jul 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marcvberg marcvberg marcvberg approved these changes

@alharris-at alharris-at alharris-at approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@AaronZyLee @marcvberg @alharris-at @sundersc @amplify-data-ci @lazpavel