Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
    Sasha Shpota
    @Shpota
    I won't be able to check your code today. I will do that tomorrow (I guess close to evening). Would that be fine?
    Jakob
    @Jakob-em
    Cool, no problem
    Inna Shpota
    @shpotainna
    I'm trying to add Vue templates :)
    Cara Sue
    @carasue
    I note nobody is working on the cli now(is any?), I am going to work on it

    I note nobody is working on the cli now(is any?), I am going to work on it

    I notice

    Sasha Shpota
    @Shpota
    @carasue 👍🏼
    Sasha Shpota
    @Shpota
    I didn't have time yesterday to complete changes of codegen.Generate(). But we can agree that we can change its signature to Generate(projectName, framework string) where framework would be one of "angular", "react", "vue". What do you think?
    Cara Sue
    @carasue
    @Shpota thus I do not have to handle the Generate() function, and leave it to u, is that right?
    Sasha Shpota
    @Shpota
    Well, I would not use a third party library to handle CLI, at least for now.
    The current CLI interface is only 16 lines of code. Adding --vue, --react and --angular would not make it much complex.
    At the same time, adding a third party lib to a Go app which is meant to be used by other developers has its implications. Apart from having to manage the dependency and it's transitive dependencies it might be that some of the dependencies require low level tools such as gcc to compile, etc (I am not sure though if https://github.com/urfave/cli requires them).
    My suggestion would be to implement it without a lib and only if the solution gets too complex use a third party tool.
    What do you think?
    Cara Sue
    @carasue
    @Shpota ok, u r right
    Cara Sue
    @carasue
    Done add arg to cli for frontend
    Sasha Shpota
    @Shpota
    Thank you! I'll review all PRs tonight.
    Cara Sue
    @carasue
    Do u think supporting Redis by default is good idea. I mean Go + React + MongoDB + Redis
    For me, a Redis is almost necessary in a project. What do u think?
    Cara Sue
    @carasue
    And should we use Trello to collaborate?
    project board is not that good to use
    I do not have the permission to edit it
    Cara Sue
    @carasue
    it also supports Github plugin
    Sasha Shpota
    @Shpota

    Do u think supporting Redis by default is good idea.

    I think supporting Redis might be an option. Though I don't think it should be by default.
    In what way do you think we can apply it in projects generated by Goxygen?
    Redis is often introduced to a web project when there is a need to cache something and all the other approaches do not lead to performance improvements. But this usually happens on later stages of a project and always require careful assessment.

    Jakob
    @Jakob-em
    I think it would add more value to add alternative databases to choose from for example an sql db instead of mongoDB
    Sasha Shpota
    @Shpota
    I agree with you. Also I've got several comments to add support for TypeScript.
    Jakob
    @Jakob-em
    Concerning the react project ?
    Sasha Shpota
    @Shpota
    Yes, at this point there is only React.
    Sasha Shpota
    @Shpota

    I have implemented this part.

    1. Changes in codegen.Generate() and templates structure.
      codegen.Generate() should be able to figure out which framework to use depending on the input parameters.

    Please take a look at the PR: Shpota/goxygen#32

    Jakob
    @Jakob-em
    Done :thumbsup:
    Sasha Shpota
    @Shpota
    Thank you! I'll apply the changes tomorrow.
    Sasha Shpota
    @Shpota
    I am working on the CLI implementation for the --frontend option. These are breaking changes and I'm going to merge them only after I check and test everything precisely. I have not included the regenerated static/generated.go file yet.
    Here is the PR: Shpota/goxygen#40
    It would be great if somebody could take a look at it and say what they think.
    Jakob
    @Jakob-em
    What about creating the generated.go file as part of the CI workflow?
    For example every time a commit is tagged with a version, the generated.go file could be generated and a new github release should be created.
    Sasha Shpota
    @Shpota
    I generally like this idea. The only concern I have is security. I am not sure how it is like with GitHub Actions, but there was time when anybody could simply run echo $ACCOUNT_SECRET in Travis and get write access to an account :)
    But it is certainly something that requires improvement. An alternative solution would be to manage templates in a separate repository and fetch them via network on the client side.
    Sasha Shpota
    @Shpota
    I have just published Goxygen v0.2.0. Thank you for your contributions!
    https://github.com/Shpota/goxygen/releases/tag/v0.2.0
    Douglas Souza
    @douglas-DS
    i've opened right now a pull request for a translation to pt-br of README file
    Lox3ur
    @cgaspart
    Hi all, when i try go get -u github.com/shpota/goxygen i have this message: # github.com/shpota/goxygen/codegen ../go/src/github.com/shpota/goxygen/codegen/codegen.go:26:13: undefined: strings.ReplaceAll
    Anthony Anonde
    @tonymj76
    hello everyone am new here so please be kind lol... am trying to build a web app with revel +vuejs wasn't easy but it worked at last... then i tried with Nuxtjs due to SEO... but i have not been able to do that with Revel... so my question is Goxygen is great but can it work with nuxtjs?
    Jakob
    @Jakob-em
    @cgaspart
    Which version of go are you using?
    Jakob
    @Jakob-em
    ReplaceAll was added with go 1.12 so you need to have at least go 1.12 installed in order to use goxygen
    @Shpota Did you test goxygen with go 1.11 bc in the readme it says that at least go 1.11 is needed but I think 1.12 is the minimal requirement
    Daniel Wolff
    @danielrwolff
    Neat project! First thing that came to mind is typescript support for React. I noticed you mentioned this is a requested feature. Can I tackle this for you?
    Jakob
    @Jakob-em
    Go for it
    Sasha Shpota
    @Shpota
    @cgaspart apparently support of Go 1.11 has been broken unintentionally. Could you please update your Go to yhe latest one? I'll fix it later.
    Sasha Shpota
    @Shpota
    @danielrwolff sure, that would be great. I have not thought yet on the best way of adding typescript. We can discuss it here. The current approach to support different technologies is prefixing file paths with the name of the technology.
    I thought about adding new databases as a next step. But I see that I need to concentrate on automations. I am planning to add the followibg functionality in the nearest time: 1) CI jobs that would build Goxygen on Go 1.11 to 1.14; 2) daily scheduled CI jobs that would generate test projects for each technology (vue, angular, react) and build a Docker images for these projects.
    Jakob
    @Jakob-em
    Cool idea to improve the CI setup of the project. But why would schedule the second job daily and not just after every commit to the master?
    Daniel Wolff
    @danielrwolff
    @Shpota I've already thrown together a "react-ts" webapp that's identical to the "react" webapp (but written in Typescript, obviously). I noticed you removed the service worker files from the react-js template. Is there a reason for this?
    Also, is there any desire to keep the frameworks up to date automatically? The version of react-js is a minor version behind the currently supported version already, and that got me thinking on what your plans were.
    Daniel Wolff
    @danielrwolff
    PR for React+Typescript is up at #51