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.

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

can't change the completeMessage text #4

Open
hiddevdploeg opened this issue Sep 2, 2018 · 16 comments
Open

can't change the completeMessage text #4

hiddevdploeg opened this issue Sep 2, 2018 · 16 comments
Assignees

Comments

@hiddevdploeg
Copy link

Hi,

First of all great job! I Love the simplicity of this.

For some reason my completeMessage is You’ve been added to our list. X instead of the text I defined as completeMessage={"Thanks for subscribing!"}

Any idea how this is possible?

@iamkevingreen
Copy link
Member

@hiddevdploeg could you share the whole rimp snippet (minus your mailchimp key). The above should work just like the other fields.

@hiddevdploeg
Copy link
Author

hiddevdploeg commented Sep 7, 2018

Sure thing! Here goes:

        <Rimp
          buttonValue={`Sign Up`}
          buttonStyles={`cta`}
          containerStyles={`forms`}
          mailChimpUrl={url}
          placeholder="Enter your email address"
          completeMessage={"Thanks for becoming a changemaker!"}
          showError={false}
        />

@iamkevingreen
Copy link
Member

@hiddevdploeg I believe the quote syntax you are using may be wrong. Could you try instead of the double quotes to use the ticks, or remove the brackets from wrapping the string? so either:

completeMessage='thanks for becoming a changemaker!'

or

completeMessage={`Thanks for becoming a changemaker!`}

@hiddevdploeg
Copy link
Author

For some reason I still get this:
image

@iamkevingreen
Copy link
Member

@hiddevdploeg can you tell me what version of rimp you have in your package.json?

@hiddevdploeg
Copy link
Author

0.1.7

@iamkevingreen
Copy link
Member

mmm... that was from the initial release of 0.1.1, i'm also using the version you listed on another site without issue. Could you try removing the package and readding it. Also check the contents of the node_module folder and ensure that the text above is not there?

https://github.com/the-couch/rimp/blob/master/package/index.js#L138 <-- this is correct

@hiddevdploeg
Copy link
Author

For some reason I did find it in the index.js and it's fixed now 👍 Thanks

@hiddevdploeg
Copy link
Author

Not sure how this happened, I did re-install it and it was still there

@iamkevingreen
Copy link
Member

I'll take a look at the npm version, but this is pretty legacy, are you using npm or yarn? @hiddevdploeg

@iamkevingreen iamkevingreen self-assigned this Sep 7, 2018
@hiddevdploeg
Copy link
Author

Yarn

@hiddevdploeg
Copy link
Author

Other than that it's brilliant. Any way I can get the GDPR things pop up or link mailchimp confirmation screen? I'm afraid it's legally not allowed to add people directly to a list, no?

@iamkevingreen
Copy link
Member

You should still get the confirm email in your inbox unless you have that disabled. The purpose for this package is to allow for clientside subscribing without the need to take users to the confirmation page. Otherwise you can't do the like clientside hey thanks for subscribing if it takes you to another page. If those concerns are real for you, you should just use the normal embed code provided by mailchimp and not do an ajax style clientside request.

This can usually only be achieved when using server side mailchimp api. However this is an undocumented solution.

Alternatively open to ways to make this more accessible!

@camsjams
Copy link

camsjams commented Nov 21, 2018

The source code on npm for version 0.1.7, has the dist.js as:

  _react2.default.createElement(
          'p',
          null,
          'You\u2019ve been added to our list. X'
        )

Rather than using this <p>{this.state.completeMessage}</p> as shown in this Git repo.

@iamkevingreen
Copy link
Member

will take another look at this @camsjams i think npm just hates me 🤔

@michaeldever
Copy link

Also seeing this issue using yarn.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants