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.

Sign up for GitHub

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

Jump to bottom

Rebranding #9

Closed
gblmarquez opened this issue Oct 6, 2020 · 14 comments
Closed

Rebranding #9

gblmarquez opened this issue Oct 6, 2020 · 14 comments
Assignees
@andrekutianski
Labels
enhancement New feature or request
Milestone
Refactoring

Comments

@gblmarquez
Copy link

@souzadavi precisamos fazer um rebranding do modulo.
@gblmarquez gblmarquez added the enhancement New feature or request label Oct 6, 2020
@souzadavi
Copy link

Alterações:

  • nome do módulo
  • descrição do módulo
  • logo do módulo
  • alterar pasta (validar se pasta antiga não existe)

gblmarquez pushed a commit that referenced this issue Jan 21, 2021
@souzadavi
Merge pull request #9 from LinkNacional/issues
68c485b
@souzadavi souzadavi mentioned this issue Jul 12, 2021
Closed
@luiznfeio luiznfeio reopened this Aug 26, 2021
@luiznfeio
Copy link
Contributor

luiznfeio commented Aug 26, 2021 •
edited by andrekutianski
Loading

Precisamos tirar esse nome gofas e padronizar para nfeio_serviceinvoice (por extenso: NFE.io - Emissão de Notas Fiscais de Serviço)

Isso inclui renomear:

  • Telas
  • Views
  • Paths / Pastas
  • Arquivos
  • Métodos e variáveis
  • Nome de classes
  • Tabelas de banco de dados

OBS importante: essa issue deve prever a migração de dados para clientes que já usam o add-on ✔️

@luiznfeio luiznfeio changed the title rebranding Rebranding Aug 26, 2021
@luiznfeio luiznfeio added this to the Refactoring milestone Aug 26, 2021
@luiznfeio luiznfeio mentioned this issue Aug 26, 2021
Closed
@andrekutianski andrekutianski self-assigned this Sep 30, 2021
@andrekutianski
Copy link
Collaborator

Definido junto com @luizdalciconfeio o novo nome do módulo Addon para: nfeio_serviceinvoices

andrekutianski added a commit that referenced this issue Oct 15, 2021
@andrekutianski
rotina de migração da configuração do módulo #9
e57c109
andrekutianski added a commit that referenced this issue Oct 16, 2021
@andrekutianski
rotina de migração das opções personalizadas de clientes #9
6543eb8
andrekutianski added a commit that referenced this issue Oct 16, 2021
@andrekutianski
rotina de migração das notas fiscais #9
8659577
andrekutianski added a commit that referenced this issue Oct 16, 2021
@andrekutianski
rotina de migração dos códigos de serviço personalizados #9
b37a2d9
andrekutianski added a commit that referenced this issue Oct 18, 2021
@andrekutianski
interface gestão de códigos de serviços #9
0c0e66a
andrekutianski added a commit that referenced this issue Oct 18, 2021
@andrekutianski
pequenas correções na interface de códigos de serviços #9
01bed86
andrekutianski added a commit that referenced this issue Oct 18, 2021
@andrekutianski
interface de gestão das notas fiscais #9
44666f0
andrekutianski added a commit that referenced this issue Oct 18, 2021
@andrekutianski
replicação das funções legadas #9
304e79e
@andrekutianski
Copy link
Collaborator

andrekutianski commented Oct 18, 2021 •
edited
Loading

Checklist de migração

Checklist com os itens primordiais a serem implementados e verificados para permitir uma migração segura e sem atritos entre a versão anterior a 2.0 (antiga estrutura).

  • webhooks: webhooks atuais devem apontar para novo endereço
  • migrar configurações: migrar as configurações atuais do antigo módulo para o novo e evitar configuração
  • migrar notas: migrar os dados atuais das notas da tabela gofasnfeio para mod_nfeio_si_serviceinvoices
  • migrar produtos: migrar os registros dos códigos de serviços personalizado dos produtos da tabela tblproductcode para mod_nfeio_si_productcode
  • migrar clientes: migrar os registros de emissão de nf personalizados dos clientes da tabela mod_nfeio_custom_configs para mod_nfeio_si_custom_configs
  • campos obrigatórios: verificar se os campos obrigatórios estão preenchidos e alertar
  • estrutura de tabelas: obedecer a estrutura atual das tabelas para menor atrito na migração
  • replicação das funções legadas: replicar todas as funções legadas existentes
  • adaptar funções legadas: realizar as mínimas e apenas necessárias adaptações e modificações nas funções legadas para a nova estruturação.
  • replicação dos gatilhos

andrekutianski added a commit that referenced this issue Oct 18, 2021
@andrekutianski
adaptação das funções legadas #9
827b6b8
andrekutianski added a commit that referenced this issue Oct 18, 2021
@andrekutianski
metodo para inicializacao de valores no campo legado issue_note_condi…
1642784 
…tions #9
@andrekutianski
Copy link
Collaborator

andrekutianski commented Oct 18, 2021 •
edited
Loading

Checklist dos hooks a serem migrados

  • InvoiceCreation
  • InvoicePaid
  • AdminInvoicesControlsOutput
  • InvoiceCancelled
  • DailyCronJob
  • AfterCronJob
  • ProductDelete
  • AdminClientProfileTabFields
  • AdminClientProfileTabFieldsSave

andrekutianski added a commit that referenced this issue Oct 18, 2021
@andrekutianski
hooks InvoiceCreation e InvoicePaid #9
b5ba58c
andrekutianski added a commit that referenced this issue Oct 18, 2021
@andrekutianski
hook InvoiceCancelled #9
ca5e09e
andrekutianski added a commit that referenced this issue Oct 18, 2021
@andrekutianski
hook DailyCronJob #9
12ea2c5
andrekutianski added a commit that referenced this issue Oct 19, 2021
@andrekutianski
hook AfterCronJob #9
53c1fdd
andrekutianski added a commit that referenced this issue Oct 19, 2021
@andrekutianski
hook ProductDelete #9
d41f5bf
andrekutianski added a commit that referenced this issue Oct 19, 2021
@andrekutianski
hook AdminClientProfileTabFieldsSave #9
ca90197
andrekutianski added a commit that referenced this issue Oct 19, 2021
@andrekutianski
hook AdminClientProfileTabFields #9
ec1f28e
andrekutianski added a commit that referenced this issue Oct 19, 2021
@andrekutianski
hook AdminInvoicesControlsOutput #9
45d1d6a
andrekutianski added a commit that referenced this issue Oct 19, 2021
@andrekutianski
funções legadas para o admin #9
62f5a02
andrekutianski added a commit that referenced this issue Oct 19, 2021
@andrekutianski
adaptação das funções legadas para o admin #9
fc47a18
@andrekutianski
Copy link
Collaborator

andrekutianski commented Oct 19, 2021 •
edited
Loading

@luizdalciconfeio o módulo na nova estrutura já está pronto para testes, verificação e validação, na branch refact_9

Para dar como completa as rotinas de migração (fora as já criadas e listadas aqui) ainda preciso criar:

  • rotina para criar um novo webhook com o novo caminho do callback
  • rotina para desativar o webhook do callback da versão anterior (para evitar problemas com módulos que não foram devidamente migrados)
  • alerta visual no admin caso a versão antiga ainda esteja ativa em paralelo com a nova

Fora as questão mencionadas a cima, já testei todas as rotinas existentes e por aqui o módulo não apresenta nenhum problema, tendo todas as rotinas em operação.

@andrekutianski
Copy link
Collaborator

Não obstante, falta testar no WHMCS 7.2 (estou trabalhando no 8.2)

This was referenced Oct 19, 2021
Closed
Closed
Closed
Closed
andrekutianski added a commit that referenced this issue Oct 28, 2021
@andrekutianski
correção do callback #9
5d1c46c
andrekutianski added a commit that referenced this issue Oct 28, 2021
@andrekutianski
atualização de nome de tabela de gofasnfeio para mod_nfeio_si_service…
6196b04 
…invoices #9
@andrekutianski
Copy link
Collaborator

@luizdalciconfeio existira alguma objeção em ter a criação do webhook da nova estrutura automaticamente na instalação/migração?

Hoje existe uma função que cria webhook, só não consegui encontrar (na antiga estrutura) referência para sua execução. Posso aproveitar ela e garantir que existirá o callback no lado da NFE.io

@andrekutianski
Copy link
Collaborator

@luizdalciconfeio existira alguma objeção em ter a criação do webhook da nova estrutura automaticamente na instalação/migração?

Hoje existe uma função que cria webhook, só não consegui encontrar (na antiga estrutura) referência para sua execução. Posso aproveitar ela e garantir que existirá o callback no lado da NFE.io

Aproveitei a lógica atual dos webhooks, efetuando algumas correções e adaptações para a nova estrutura.

@andrekutianski
Copy link
Collaborator

andrekutianski commented Nov 1, 2021 •
edited
Loading

Iniciando documentação da nova estrutura.

  • Documentação de migração/atualização
  • Documentação de instalação
  • Documentação de configuração

andrekutianski added a commit that referenced this issue Nov 1, 2021
@andrekutianski
refactor(core): adaptado rotinas de webhook #9
8482f2a
@andrekutianski
Copy link
Collaborator

Documentação de atualização/migração concluída e disponível em https://nfe.github.io/whmcs-addon/docs/atualizacao

@andrekutianski
Copy link
Collaborator

Documentação de instalação concluída e disponível em https://nfe.github.io/whmcs-addon/docs/instalacao/

@andrekutianski
Copy link
Collaborator

Documentação de configuração concluída e disponível em https://nfe.github.io/whmcs-addon/docs/configuracao/

@andrekutianski
Copy link
Collaborator

andrekutianski commented Nov 22, 2021 •
edited
Loading

branch para a release 2.0.0 pronta em https://github.com/nfe/whmcs-addon/tree/release-2.0.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andrekutianski andrekutianski

Labels
enhancement New feature or request
Projects
None yet
Milestone
Refactoring
Development

No branches or pull requests
4 participants
@gblmarquez @souzadavi @andrekutianski @luiznfeio