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

Adonis V5 migration #50

Closed
5 tasks done
HigoRibeiro opened this issue Jan 19, 2021 · 10 comments
Closed
5 tasks done

Adonis V5 migration #50

HigoRibeiro opened this issue Jan 19, 2021 · 10 comments

Comments

@HigoRibeiro
Copy link
Contributor

HigoRibeiro commented Jan 19, 2021

We're working on integration with adonis v5.

🚧 Roadmap

Special thanks @RodolfoSilva for initializing the implementation in PR #39

@ruanitto
Copy link

Tava fazendo alguns testes já com o pacote, não sei se vocês já realizaram o ajuste, caso sim pode desconsiderar.

Mas ele dá um erro com a conexão ao redis, pq nas linhas 40 e 116, ele passa o objeto BullConnectionContract inteiro, e não apenas a configuração definida.

@fhugoduarte
Copy link

fhugoduarte commented Jan 20, 2021

Tava fazendo alguns testes já com o pacote, não sei se vocês já realizaram o ajuste, caso sim pode desconsiderar.

Mas ele dá um erro com a conexão ao redis, pq nas linhas 40 e 116, ele passa o objeto BullConnectionContract inteiro, e não apenas a configuração definida.

Hi @ruanitto 🤗

We are making some changes, but could you give us more information about the error?

@zulfi007
Copy link

Finally🤓

@ashokgelal
Copy link

ashokgelal commented Jan 20, 2021

Tava fazendo alguns testes já com o pacote, não sei se vocês já realizaram o ajuste, caso sim pode desconsiderar.

Mas ele dá um erro com a conexão ao redis, pq nas linhas 40 e 116, ele passa o objeto BullConnectionContract inteiro, e não apenas a configuração definida.

Sorry I don't talk Portuguese and I'm just trying to help :)

I don't know if this is related but some of the instructions in the README file is wrong/incomplete about how to configure the connection. I've updated the README in fork so feel free to consult it: https://github.com/ashokgelal/adonis-bull

Basically, there are two steps missing/incomplete in the README:

  1. Setup: node ace invoke @ashokgelal/adonis-bull
  2. config/bull.ts:
import Env from '@ioc:Adonis/Core/Env'
import { BullConfig } from '@ioc:Rocketseat/Bull'


const bullConfig: BullConfig = {
  connection: Env.get('BULL_CONNECTION', 'bull'),
  connections: {
    bull: {
  	  host: Env.get('BULL_REDIS_HOST'),
  	  port: Env.get('BULL_REDIS_PORT'),
  	  password: Env.get('BULL_REDIS_PASSWORD', ''),
  	  db: 0,
  	  keyPrefix: '',
    },
  }
}

export default bullConfig

Happy to help you on the Adonis discord server if you need more help.

@fhugoduarte
Copy link

fhugoduarte commented Jan 20, 2021

Tava fazendo alguns testes já com o pacote, não sei se vocês já realizaram o ajuste, caso sim pode desconsiderar.
Mas ele dá um erro com a conexão ao redis, pq nas linhas 40 e 116, ele passa o objeto BullConnectionContract inteiro, e não apenas a configuração definida.

Sorry I don't talk Portuguese and I'm just trying to help :)

I don't know if this is related but some of the instructions in the README file is wrong/incomplete about how to configure the connection. I've updated the README in fork so feel free to consult it: https://github.com/ashokgelal/adonis-bull

Basically, there are two steps missing/incomplete in the README:

  1. Setup: node ace invoke @ashokgelal/adonis-bull
  2. config/bull.ts:
import Env from '@ioc:Adonis/Core/Env'
import { BullConfig } from '@ioc:Rocketseat/Bull'


const bullConfig: BullConfig = {
  connection: Env.get('BULL_CONNECTION', 'bull'),
  connections: {
    bull: {
  	  host: Env.get('BULL_REDIS_HOST'),
  	  port: Env.get('BULL_REDIS_PORT'),
  	  password: Env.get('BULL_REDIS_PASSWORD', ''),
  	  db: 0,
  	  keyPrefix: '',
    },
  }
}

export default bullConfig

Happy to help you on the Adonis discord server if you need more help.

No problem, we're talking in English by default, as this package isn't only used by people who speak Portuguese.

Thx you for your report, but we added all the necessary setup for the bull configuration in the invoke command. Then with the invoke command your project will be ready to work with jobs that will be created with the make:job command. We're done with that. So we hope you don't need to do any manual configuration on your project.

@ruanitto
Copy link

Tava fazendo alguns testes já com o pacote, não sei se vocês já realizaram o ajuste, caso sim pode desconsiderar.
Mas ele dá um erro com a conexão ao redis, pq nas linhas 40 e 116, ele passa o objeto BullConnectionContract inteiro, e não apenas a configuração definida.

Sorry I don't talk Portuguese and I'm just trying to help :)

I don't know if this is related but some of the instructions in the README file is wrong/incomplete about how to configure the connection. I've updated the README in fork so feel free to consult it: https://github.com/ashokgelal/adonis-bull

Basically, there are two steps missing/incomplete in the README:

  1. Setup: node ace invoke @ashokgelal/adonis-bull
  2. config/bull.ts:
import Env from '@ioc:Adonis/Core/Env'
import { BullConfig } from '@ioc:Rocketseat/Bull'


const bullConfig: BullConfig = {
  connection: Env.get('BULL_CONNECTION', 'bull'),
  connections: {
    bull: {
  	  host: Env.get('BULL_REDIS_HOST'),
  	  port: Env.get('BULL_REDIS_PORT'),
  	  password: Env.get('BULL_REDIS_PASSWORD', ''),
  	  db: 0,
  	  keyPrefix: '',
    },
  }
}

export default bullConfig

Happy to help you on the Adonis discord server if you need more help.

Sorry, my English is not very good, I ended up using the google translator but at the time it was kind of curled up and I ended up writing in Portuguese.
First of all, thank you very much for teaching all of you with the package, I think it is sensational.

I did all the configuration really according to your repository, but the connection to the redis failed, I performed the debug, and in the BullManager.ts file, there was an error when passing the connections to Bull, but it has already been fixed.
Thanks :)

@zulfi007
Copy link

My suggestion is keep migration in alpha branch and in typescript as adonis v5 is mostly in ts.
Readme file also needs rewrite as per new rules.

@jpedroschmitz
Copy link
Contributor

Hey @zulfi007, we will do that. The documentation and code for version 5 will be on a separated branch (named alpha) while Adonis 5 is still in beta. After that, we will merge it on master and will be the default.

@jpedroschmitz
Copy link
Contributor

jpedroschmitz commented Jan 27, 2021

Hey guys, the support for version 5 was officially released 🎉

It is a premajor, and may contain bugs, but we encourage you to use it on your projects and report any problem you may have.

To get started with it, you can install it with NPM or Yarn:

yarn add @rocketseat/adonis-bull@alpha
# or
npm install @rocketseat/adonis-bull@alpha

After installing it, run node ace invoke @rocketseat/adonis-bull, choose a start option and you're ready to go ⚡️

You can find the complete documentation here, and if you want to add more examples or guides, don't hesitate to open a PR!

@LeCoupa
Copy link
Contributor

LeCoupa commented Jan 27, 2021

Great @jpedroschmitz! Will install it tomorrow :)

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

No branches or pull requests

7 participants