Merge pull request #130 from Algoteers/feature/client-update-notifications
Feature/client update notifications
This commit is contained in:
5 files changed
+8
-4
No files matched your search
@@ -5,6 +5,10 @@ All notable changes to this project will be documented in this file.
|
||||
### Changed
|
||||
- [List changes here for upcoming release]
|
||||
|
||||
## 0.1.3 - 2018-12-17
|
||||
### Change
|
||||
- Client: Fix newsletter signup notification integration
|
||||
|
||||
## 0.1.2 - 2018-12-16
|
||||
### Change
|
||||
- Client: Upgrade Material-ui to fix webpack deepmerge import bug
|
||||
|
||||
+1
-1
@@ -1,6 +1,6 @@
|
||||
{
|
||||
"name": "master-app-client",
|
||||
"version": "0.0.1",
|
||||
"version": "0.1.3",
|
||||
"description": "Responsible for loading all web modules and running them.",
|
||||
"main": "src/index.js",
|
||||
"scripts": {
|
||||
|
||||
@@ -122,7 +122,7 @@ export class SignupForm extends Component {
|
||||
console.log('NewsletterSignup handleSubmit: ', this.state.email)
|
||||
const signupResponse = await NewsletterSignup({ variables: { email: this.state.email } })
|
||||
console.log('NewsletterSignup: ', signupResponse)
|
||||
if (signupResponse.data.master_NewsletterSignup === 'SUCCESS') this.setState({ email: '', success: true })
|
||||
if (signupResponse.data.notifications_Master_NewsletterSignup === 'SUCCESS') this.setState({ email: '', success: true })
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
@@ -56,7 +56,7 @@ export class SignupVerifyResponse extends React.Component {
|
||||
}
|
||||
if (data !== undefined && data !== null) {
|
||||
console.log(data)
|
||||
if (data.master_NewsletterSignupVerify === 'SUCCESS') {
|
||||
if (data.notifications_Master_NewsletterSignupVerify === 'SUCCESS') {
|
||||
return (
|
||||
<React.Fragment>
|
||||
<BannerTopBar
|
||||
|
||||
+1
-1
@@ -1,6 +1,6 @@
|
||||
{
|
||||
"name": "master-app",
|
||||
"version": "0.1.1",
|
||||
"version": "0.1.3",
|
||||
"description": "Responsible for loading all web modules and running them.",
|
||||
"main": "index.js",
|
||||
"scripts": {
|
||||
|
||||
Reference in new issue
Block a user