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

add cancel to delete and minor readme/help text changes #2

Merged
merged 1 commit into from
Aug 23, 2018
Merged

add cancel to delete and minor readme/help text changes #2

merged 1 commit into from
Aug 23, 2018

Conversation

UnleashedMind
Copy link
Contributor

Issue #, if available:

Description of changes:

  • add Cancel option to the amplify notifications delete
  • minor readme/help text changes

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

kaustavghosh06
kaustavghosh06 approved these changes Aug 23, 2018
View reviewed changes
Copy link
Contributor

@kaustavghosh06 kaustavghosh06 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@kaustavghosh06 kaustavghosh06 merged commit 0d264ae into aws-amplify:master Aug 23, 2018
@chrisco512 chrisco512 mentioned this pull request Sep 6, 2018
Closed
@CodySwannGT CodySwannGT mentioned this pull request Oct 4, 2018
Closed
@GSynergy GSynergy mentioned this pull request Nov 3, 2018
Closed
@mikeparisstuff mikeparisstuff mentioned this pull request Mar 19, 2019
Closed
@janhesters janhesters mentioned this pull request Mar 21, 2019
Closed
@ElliottBabb ElliottBabb mentioned this pull request May 6, 2019
Closed
yuth added a commit that referenced this pull request Jun 3, 2019
@yuth
Fix integration tests #2
15f98fa
@karldanninger karldanninger mentioned this pull request Jun 25, 2019
Closed
kaustavghosh06 referenced this pull request in kaustavghosh06/amplify-cli Sep 9, 2019
@garyleefight
Merge pull request #2 from kaustavghosh06/cliinteg
da61728 
hook up hosting console
@rrrix rrrix mentioned this pull request Nov 14, 2019
Closed
attilah pushed a commit that referenced this pull request Dec 3, 2019
@yuth @attilah
fix(amplify-codegen-appsync-model-plugin): use LazyCollection (JS/TS) (…
4fa3a53 
…#2905)

* fix(amplify-codegen-appsync-model-plugin): use LazyCollection (JS/TS)

Connected models to use LazyCollection in the generated Java Script and TypeScript code

* address review commet

* address review commet #2
@malcomm malcomm mentioned this pull request Feb 5, 2020
Closed
@nagey nagey mentioned this pull request Mar 19, 2020
Closed
@drixta drixta mentioned this pull request Apr 29, 2020
Closed
@ncohen17 ncohen17 mentioned this pull request May 14, 2020
Closed
@asmajlovicmars asmajlovicmars mentioned this pull request Oct 23, 2020
Closed
@chantlong chantlong mentioned this pull request Nov 6, 2020
Closed
@PeteDuncanson PeteDuncanson mentioned this pull request Nov 27, 2020
Closed
yuth added a commit that referenced this pull request Dec 2, 2020
@yuth
chore: fix merge error #2
c5bdbe1
@edwardfoyle edwardfoyle mentioned this pull request Dec 30, 2020
Closed
@MFranca MFranca mentioned this pull request Dec 30, 2020
Open
@bishonbopanna bishonbopanna mentioned this pull request Mar 23, 2021
Closed
4 tasks
SwaySway pushed a commit that referenced this pull request May 20, 2021
@yuth
feat: prep work for Cognito SMS Sandbox #2 (#7338)
3dbb3bf 
* feat: display SNS sandbox status

* chore: SNS API error handling

* chore: add support for SMS Sandbox check

* chore: auth add show sandbox warning

* chore: add some tests

* chore: address review comments and remove unused code
@github-actions github-actions bot added the referenced-in-release Issues referenced in a published release changelog label May 21, 2021
@aws-amplify aws-amplify deleted a comment from github-actions bot May 24, 2021
@siegerts siegerts removed the referenced-in-release Issues referenced in a published release changelog label May 24, 2021
SwaySway pushed a commit that referenced this pull request May 25, 2021
@abhi7cr
Merge pull request #2 from aws-amplify/master
908d192 
fix: e2e failed tests for auth Triggers PR (#7262)
@arkllc arkllc mentioned this pull request Jun 12, 2021
Closed
@dorontal dorontal mentioned this pull request Jul 5, 2021
Closed
4 tasks
cjihrig pushed a commit to ctjlewis/amplify-cli that referenced this pull request Jul 12, 2021
@yuth @cjihrig-aws
feat: prep work for Cognito SMS Sandbox aws-amplify#2 (aws-amplify#7338)
b358610 
* feat: display SNS sandbox status

* chore: SNS API error handling

* chore: add support for SMS Sandbox check

* chore: auth add show sandbox warning

* chore: add some tests

* chore: address review comments and remove unused code
@paddlefish paddlefish mentioned this pull request Aug 17, 2021
Closed
4 tasks
ammarkarachi pushed a commit to ammarkarachi/amplify-cli that referenced this pull request Oct 22, 2021
@jhockett
Test e2e (aws-amplify#2)
df1f75a
ammarkarachi pushed a commit to ammarkarachi/amplify-cli that referenced this pull request Oct 23, 2021
@jhockett
Test e2e (aws-amplify#2)
cf57eef
ammarkarachi pushed a commit to ammarkarachi/amplify-cli that referenced this pull request Oct 25, 2021
@jhockett
Test e2e (aws-amplify#2)
cc23a3d
yuth added a commit that referenced this pull request Nov 6, 2021
@yuth
build: dependecy update to use the latest version of packages in mono…
62d0719 
…repo #2
@ghost ghost mentioned this pull request Feb 9, 2022
Open
2 tasks
@slaat slaat mentioned this pull request May 17, 2022
Closed
@jevakallio jevakallio mentioned this pull request Mar 22, 2022
Closed
5 tasks
@BBopanna BBopanna mentioned this pull request Jul 25, 2022
Closed
5 tasks
danielleadams pushed a commit that referenced this pull request Aug 11, 2022
@ykethan
Merge pull request #2 from ykethan/APILambdaTemplates
e723f4b
@valentinrelief valentinrelief mentioned this pull request Sep 15, 2022
Open
5 tasks
@OperationalFallacy OperationalFallacy mentioned this pull request Jan 11, 2024
Closed
abhi7cr pushed a commit that referenced this pull request Aug 29, 2024
@Sanayshah2
Merge pull request #2 from sdstolworthy/merge-sanay-migrations
6dae70d 
fix: remove incorrect merge resolution
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kaustavghosh06 kaustavghosh06 kaustavghosh06 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.
3 participants
@UnleashedMind @kaustavghosh06 @siegerts