Adiciona modelo VisualResourceBase e testes #937
Conversation
| @@ -0,0 +1,46 @@ | |||
| from packtools.sps.models.accessibility_data import AccessibilityData | |||
There was a problem hiding this comment.
@Rossi-Luciano AccessibilityData já está no branch master?
There was a problem hiding this comment.
@robertatakenaka, tem um PR específico para AccessibilityData
There was a problem hiding this comment.
@Rossi-Luciano então este está aprovado, mas você faça o merge quando o outro estiver aprovado
|
|
||
|
|
||
| class XmlVisualResource: | ||
| RESOURCE_TYPES = [] |
There was a problem hiding this comment.
@Rossi-Luciano como se instancia? como se usa?
There was a problem hiding this comment.
@robertatakenaka modifiquei para RESOURCE_TYPES ser um parâmetro.
| for node in self.article_and_subarticles: | ||
| full_text = Fulltext(node) | ||
|
|
||
| for resource_type, resource_class in self.RESOURCE_TYPES: |
There was a problem hiding this comment.
@Rossi-Luciano entendi o uso, mas acho que não tem demanda para isso. Está sendo misturado no retorno inline-media e media. Acho mais conveniente haver um método para cada uma, caso haja a demanda de obter ambas, que cada método seja chamado
There was a problem hiding this comment.
@robertatakenaka modifiquei para RESOURCE_TYPES ser um parâmetro.
O que esse PR faz?
Adiciona modelo VisualResourceBase e testes.
Onde a revisão poderia começar?
Por commit.
Como este poderia ser testado manualmente?
NA
Algum cenário de contexto que queira dar?
NA
Screenshots
NA
Quais são tickets relevantes?
Substitui PR #915
Referências
NA