https://supertokens.com/ logo
Channels
Powered by
#support-questions
  • r

    rp

    11/14/2021, 6:45 PM
    We will think about how to make the interfaces design better so that stuff like this is possible without copy / pasting the code.
  • r

    rp

    11/14/2021, 6:45 PM
    I'll give you what to copy paste.. 2 mins
  • r

    rp

    11/14/2021, 6:56 PM
    Okay so this is rather long.. but here it goes
  • r

    rp

    11/14/2021, 6:57 PM
  • r

    rp

    11/14/2021, 6:59 PM
    The post sign up logic can come: - https://gist.github.com/rishabhpoddar/07de224632f32ea78b58b1203426dd57#file-customsigninuppost-ts-L40; and - https://gist.github.com/rishabhpoddar/07de224632f32ea78b58b1203426dd57#file-customsigninuppost-ts-L99 (if 
    response.createdNewUser === true
    ) To insert the custom fields in the session, you can add them when calling the 
    createNewSession
    function.
  • r

    rp

    11/14/2021, 7:09 PM
    You will need to import 
    "axios"
    and 
    "qs"
  • m

    mpicciolli

    11/14/2021, 7:10 PM
    Yes no problem. Thank you ! I ma going to try
  • m

    mpicciolli

    11/14/2021, 11:03 PM
    It works 🥳 . Thanks @User
  • r

    rp

    11/15/2021, 2:47 AM
    Ok great!!
  • t

    tdfirth

    11/16/2021, 7:59 PM
    Hey guys, hope you're well
  • t

    tdfirth

    11/16/2021, 7:59 PM
    I've been getting set up with supertokens and so far it's great. I've just run into a weird issue though
  • t

    tdfirth

    11/16/2021, 8:00 PM
    I am using vite to build my project and everything works beautifully in dev mode, but my production builds lead to a runtime error.
  • t

    tdfirth

    11/16/2021, 8:01 PM
    The error I am seeing is:
  • t

    tdfirth

    11/16/2021, 8:01 PM
    Copy code
    TypeError: cd.default.getInstanceOrThrow is not a function
  • t

    tdfirth

    11/16/2021, 8:01 PM
    I have create a minimal repo that replicates the issue for me here:
  • t

    tdfirth

    11/16/2021, 8:01 PM
  • t

    tdfirth

    11/16/2021, 8:01 PM
    Been wrestling with this for some time 😅 , any thoughts would be much appreciated!
  • t

    tdfirth

    11/16/2021, 8:02 PM
    To replicate, you should just have to run 
    npm build && npm serve
  • t

    tdfirth

    11/16/2021, 8:02 PM
    running the dev server with 
    npm dev
    is fine
  • p

    porcellus

    11/16/2021, 8:03 PM
    Hi, I'll check it out in a bit 🙂
  • t

    tdfirth

    11/16/2021, 8:03 PM
    Hey! yeah no rush
  • t

    tdfirth

    11/16/2021, 8:03 PM
    I think it's something to do with this:
  • t

    tdfirth

    11/16/2021, 8:04 PM
    Copy code
    import * as reactRouterDom from "react-router-dom";
...
        {getSuperTokensRoutesForReactRouterDom(reactRouterDom)}
  • t

    tdfirth

    11/16/2021, 8:05 PM
    You use 
    require("react-router-dom")
    in the docs but that won't work with vite as I understand it (though my js knowledge is .... mediocre)
  • p

    porcellus

    11/16/2021, 8:56 PM
    So that's not the cause of the error, it's something else, but very strange.
  • p

    porcellus

    11/16/2021, 9:00 PM
    So it doesn't get to the part where it'd use 
    reactRouterDom
    it tries to call 
    cd.default.getInstanceOrThrow
    which is undefined, while 
    cd.default.default.getInstanceOrThrow
    is defined (and points to the right thing)
  • p

    porcellus

    11/16/2021, 9:02 PM
    The strange part is the 
    default.default
    part, because it looks like it was wrapped twice for some reason. I tried a quick search to see if it was a bug in the bundler, but nothing came up.
  • p

    porcellus

    11/16/2021, 9:21 PM
    Yeah, so I've found the problem: we have a circular dependency (which is bad, but worked so far), and the interaction between vite build and our tsc output goes wrong... (I can provide more details if you are interested).
  • p

    porcellus

    11/16/2021, 9:40 PM
    I'll try and come up with a quick solution tomorrow, it's likely that we can solve this by publishing es modules as well (although removing circular deps would be better). You could raise this issue on github if you want to track it 🙂
  • t

    tdfirth

    11/17/2021, 8:18 AM
    Hey @User thanks so much for looking into it
1...160161162...343Latest