-
Notifications
You must be signed in to change notification settings - Fork 5
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
Join fields can't be added to the signup form #7
Comments
lodash is always present in our front end build, unless you're using the
lean option, which apostrophe signup does not yet support natively. As for
the joins, there is indeed a fair amount of JavaScript that we don't
automatically push to anonymous site visitors because of the weight of it
on every page. however you can bring those features in using the scene:
user option for a custom page type.
…On Wed, Sep 11, 2019, 9:03 AM Jose96GIT ***@***.***> wrote:
Hi, I've tried to add some fields with joinByOne or joinByArray types and
when I load the signup page, the browser show me some errors in its
console. And those buttons are not working. Is this maybe an issue to fix?
I'll attach an image with the browser error and also a simple project that
reflects this.
This is not related with the issue, but I also noticed that there's no a
require of *lodash*, and it's necessary to do the password shallow copy.
Thanks!
[image: image]
<https://user-images.githubusercontent.com/18165550/64685033-8430af00-d486-11e9-9dcd-8e0f86aff030.png>
test-signup-project.zip
<https://github.com/apostrophecms/apostrophe-signup/files/3599980/test-signup-project.zip>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#7>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAAH27OMSLOUSDA3G4RYYCDQJDUC5ANCNFSM4IVTXFYA>
.
|
Hi Tom, what i was refearing with lodash is that there isn't any require of it, so if I'm not understanding it wrong, Apostrophe can't know that _ is referencing lodash. In relation to the joins, by default this module is setting the scene to user, eventhough is not pushing all the necessary Javascript. Is there something that I'm forgetting to do? I've also prove it in homepage, setting the req.scene to user, adding this code to the html.
And it also fails, so it seems that there's something that I'm not doing. Thanks for your help once again! |
This is browser code you're referring to, yes? Sometimes developers who
have always used modern webpack or browserify builds don't realize that
these libraries are sometimes pushed right into the browser and attached
directly to `window` so you don't need `require`. This is an older build
where that is the case.
…On Thu, Sep 12, 2019 at 4:47 AM Jose96GIT ***@***.***> wrote:
Hi Tom, what i was refearing with lodash is that there isn't any require
of it, so if I'm not understanding it wrong, Apostrophe can't know that
*_* is referencing lodash.
In relation to the joins, by default this module is setting the scene to
user, eventhough is not pushing all the necessary Javascript. Is there
something that I'm forgetting to do?
I've also prove it in homepage, setting the req.scene to user, adding this
code to the html.
{{ buttons.inContext('edit Image', {
url: '#',
action: 'manage-apostrophe-image'
}) }}
And it also fails, so it seems that there's something that I'm not doing.
Thanks for your help once again!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#7>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAAH27LNBYW7KZHOG6JVZMLQJH6ZNANCNFSM4IVTXFYA>
.
--
Chief Software Architect
Apostrophe Technologies
Pronouns: he / him / his
|
Extending this further would probably need to be an enterprise support
project.
…On Thu, Sep 12, 2019 at 8:22 AM Tom Boutell ***@***.***> wrote:
This is browser code you're referring to, yes? Sometimes developers who
have always used modern webpack or browserify builds don't realize that
these libraries are sometimes pushed right into the browser and attached
directly to `window` so you don't need `require`. This is an older build
where that is the case.
On Thu, Sep 12, 2019 at 4:47 AM Jose96GIT ***@***.***>
wrote:
> Hi Tom, what i was refearing with lodash is that there isn't any require
> of it, so if I'm not understanding it wrong, Apostrophe can't know that
> *_* is referencing lodash.
>
> In relation to the joins, by default this module is setting the scene to
> user, eventhough is not pushing all the necessary Javascript. Is there
> something that I'm forgetting to do?
>
> I've also prove it in homepage, setting the req.scene to user, adding
> this code to the html.
>
> {{ buttons.inContext('edit Image', {
> url: '#',
> action: 'manage-apostrophe-image'
> }) }}
>
>
> And it also fails, so it seems that there's something that I'm not doing.
>
> Thanks for your help once again!
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <#7>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAAH27LNBYW7KZHOG6JVZMLQJH6ZNANCNFSM4IVTXFYA>
> .
>
--
Chief Software Architect
Apostrophe Technologies
Pronouns: he / him / his
--
Chief Software Architect
Apostrophe Technologies
Pronouns: he / him / his
|
Hi, I've tried to add some fields with joinByOne or joinByArray types and when I load the signup page, the browser show me some errors in its console. And those buttons are not working. Is this maybe an issue to fix?
I'll attach an image with the browser error and also a simple project that reflects this.
This is not related with the issue, but I also noticed that there's no a require of lodash, and it's necessary to do the password shallow copy.
Thanks!
test-signup-project.zip
The text was updated successfully, but these errors were encountered: