plonegovbr / brasil.gov.agenda

Pacote de Agenda para Membros do Governo Brasileiro
https://plone.org.br/gov/
3 stars 2 forks source link

Compromissos de uma agenda diária podem ter a mesma data e hora #69

Open idgserpro opened 7 years ago

idgserpro commented 7 years ago

selecao_006

Deveria haver alguma validação indicando que os horários não podem ser os mesmos para um mesmo dia? Alguém sabe de algum exemplo da vida real em que isso poderia acontecer para não ter essa validação?

idgserpro commented 7 years ago

Exemplo para implementação: https://github.com/plonegovbr/brasil.gov.agenda/blob/e3bd95099bd7f500c75449bf2fead6246d6ce0af/src/brasil/gov/agenda/content/agendadiaria.py#L104 ao invés de "unicidade de agenda diária" seria "unicidade de data e hora de compromissos de uma agenda diária".

claytonc commented 7 years ago

idgserpro,

Já tem um tempo que queria perguntar sempre esquecia.

Por que o produto brasil.gov.agenda é instalado por padrão nos portais e não é um produto opcional como o brasil.gov.facetada ?

Pergunto, porque já fiz algumas instalações em alguns órgãos e ninguém usa ou já possui um sistema de agenda próprio.

Outra coisa é o bug de indexação aonde o produto de agenda força a versão 1.0 do plone.app.contenttypes < 1.1a1, que só indexa id, título e descrição.

Versão 1.0 - https://github.com/plone/plone.app.contenttypes/blob/1.0/plone/app/contenttypes/indexers.py#L43-L49

Versão - 1.1.1 - https://github.com/plone/plone.app.contenttypes/blob/1.1.1/plone/app/contenttypes/indexers.py#L37-L60

abraços,

Clayton

2017-01-20 11:47 GMT-02:00 idgserpro notifications@github.com:

Exemplo para implementação: https://github.com/plonegovbr/ brasil.gov.agenda/blob/e3bd95099bd7f500c75449bf2fead6 246d6ce0af/src/brasil/gov/agenda/content/agendadiaria.py#L104 ao invés de "unicidade de agenda diária" seria "unicidade de data e hora de compromissos de uma agenda diária".

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/plonegovbr/brasil.gov.agenda/issues/69#issuecomment-274075583, or mute the thread https://github.com/notifications/unsubscribe-auth/ABo88xgnP69ZUseYja8ncg1yNrW9Hztpks5rULrxgaJpZM4Lohaw .

hvelarde commented 7 years ago

@claytonc por favor abra issues diferentes para resolver essas questões.

idgserpro commented 7 years ago

Por que o produto brasil.gov.agenda é instalado por padrão nos portais e não é um produto opcional como o brasil.gov.facetada ?

Nao fomos nós que definimos isso, mas acredito ser devido à lei da transparência e de orientações a respeito: http://www.acessoainformacao.gov.br/lai-para-sic/sic-apoio-orientacoes/guias-e-orientacoes/guia_secaositios-sem-logo.pdf. O busca multifacetada é útil mas não imprescindível.

Pergunto, porque já fiz algumas instalações em alguns órgãos e ninguém usa ou já possui um sistema de agenda próprio.

Acho que aí é questão de preferência: atendendo as orientações de transparência, o órgão decide se usa o recurso do Portal Padrão ou não. A idéia é que o Portal Padrão forneça um mínimo denominador comum de recursos importantes incluídos para o maior número de pessoas e órgãos na instalação padrão.

Muitos portais hoje utilizam esse recurso do Agenda, como Ministério da Fazenda, Receita Federal e Presidência da República.

Outra coisa é o bug de indexação aonde o produto de agenda força a versão 1.0 do plone.app.contenttypes < 1.1a1, que só indexa id, título e descrição.

Sim, sabemos disso e já temos issue aberto. Mas quando mudamos a versão os testes quebram e precisam ser consertados. Ver https://github.com/plonegovbr/brasil.gov.agenda/issues/60 e https://github.com/plonegovbr/brasil.gov.portal/issues/240. Tem de se analisar também se não será necessário um upgradeStep devido a essa mudança de versão. Se souber consertar, recomendamos a criação de um fork para correção e a criação de um pull request. Também é de nosso interesse simplificar ao máximo essas pinagens, se possível, usar as pinagens padrões da versão do Plone utilizada, mas não podemos deixar os testes quebrados.

idgserpro commented 7 years ago

@hvelarde acabou que respondi :laughing:

@claytonc como o @hvelarde disse, se ainda tiver dúvidas, abra issues específicas.