I would like to translate my signup forms but I cannot to do that at "admin/config/services/mailchimp/signup/newsletter_signup/translate". I got "Access denied", "You are not authorized to access this page.". I tried to do this as "super admin" (uid=1).

Comments

asrob created an issue. See original summary.

asrob’s picture

Issue summary: View changes
amytswan’s picture

Category: Bug report » Feature request

This would be a good feature to add, @asrob - if you are inspired, please feel free to write a patch and we'll check it out!

asrob’s picture

Okay, I'll try to write a patch! :)

quiron’s picture

Hi guys,

I have the same problem, I writted this patch, for me is working.

Can you try it and give some feedback?

greg boggs’s picture

Thanks Quiron!

f0ns’s picture

I have the same issue, when I try to translate the signup form on admin/config/services/mailchimp/signup I get a 403 (access denied).

I Tried the patch but it doesn't seem to solve this bug.

I'll try to look into this and investigate it further in the following days.

f0ns’s picture

StatusFileSize
new1.88 KB

It took me a little longer then expected but here is the patch. Now everything is translatable for the signup form:

  • Title
  • Description
  • Submit button text
  • Confirmation message
f0ns’s picture

Status: Active » Needs review
f0ns’s picture

You can see it in action on my personal website (the block is under the body):

Dutch: https://fonsvandamme.be/blog/drupal-8-contrib-module-readremaining
English: https://fonsvandamme.be/en/blog/drupal-8-contrib-module-readremaining

robin.ingelbrecht’s picture

Status: Needs review » Reviewed & tested by the community

This patch seems to work

f0ns’s picture

Can this be pushed to the dev branch please?

brtamas’s picture

StatusFileSize
new1000 bytes

I have made the mergvars translatable.
Patch attached.

bsztreha’s picture

#8 worked for me, thanks!

luksak’s picture

#8 Works for me as well!

luksak’s picture

Also test #13 and it works as well. Worth mentioning is that this is not translated using config translation but using string translation. Should we change that?

greg boggs’s picture

Went ahead with string translation because I don't believe mergevars are Drupal Config.

  • Greg Boggs committed 8ddc346 on 8.x-1.x authored by brtamas
    Issue #2772709 by Fons Vandamme, brtamas, quiron, Greg Boggs: Cannot...
  • Greg Boggs committed de65736 on 8.x-1.x authored by brtamas
    Issue #2772709 by Fons Vandamme, brtamas, quiron: Cannot translate...
greg boggs’s picture

Status: Reviewed & tested by the community » Fixed
greg boggs’s picture

  • Greg Boggs committed 84ca073 on 8.x-1.x authored by Fons Vandamme
    Issue #2772709 by Fons Vandamme, brtamas, quiron: Cannot translate...
  • Greg Boggs committed cf281e0 on 8.x-1.x
    Revert "Issue #2772709 by Fons Vandamme, brtamas, quiron: Cannot...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.