Skip to content

Add enabled boolean field to CrossmarkPage#125

Draft
Copilot wants to merge 3 commits intomasterfrom
copilot/add-enabled-boolean-field
Draft

Add enabled boolean field to CrossmarkPage#125
Copilot wants to merge 3 commits intomasterfrom
copilot/add-enabled-boolean-field

Conversation

Copy link
Contributor

Copilot AI commented Mar 9, 2026

O que esse PR faz?

Adiciona o campo enabled (BooleanField, default=False) ao modelo CrossmarkPage para indicar se a adoção do Crossmark está ativa ou não. O valor padrão False indica que o Crossmark não está adotado, devendo ser explicitamente habilitado.

class CrossmarkPage(Document):
    doi = StringField(required=True)
    is_doi_active = BooleanField(required=True, default=True)
    enabled = BooleanField(required=True, default=False)  # novo campo
    ...

O campo também foi adicionado aos indexes do modelo.

Onde a revisão poderia começar?

opac_schema/v1/models.py — classe CrossmarkPage (~linha 1071).

Como este poderia ser testado manualmente?

from opac_schema.v1.models import CrossmarkPage
# default False
doc = CrossmarkPage(doi="10.1590/x", language="en", journal=journal)
assert doc.enabled is False

# habilitado explicitamente
doc2 = CrossmarkPage(doi="10.1590/x", language="en", journal=journal, enabled=True)
assert doc2.enabled is True

Algum cenário de contexto que queira dar?

Necessário para que sistemas consumidores do schema possam distinguir periódicos que adotaram o Crossmark daqueles que não adotaram, sem depender de lógica externa. O padrão False garante que a adoção seja explícita.

Screenshots

N/A

Quais são tickets relevantes?

Adicione a CrossmarkPage o campo enabled como boolean

Referências

N/A

Original prompt

This section details on the original issue you should resolve

<issue_title>Adicione a CrossmarkPage o campo enabled como boolean</issue_title>
<issue_description>Adicione a CrossmarkPage o campo enabled como boolean para indicar adoção ou não</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: robertatakenaka <505143+robertatakenaka@users.noreply.github.com>
Copilot AI changed the title [WIP] Add enabled boolean field to CrossmarkPage Add enabled boolean field to CrossmarkPage Mar 9, 2026
class CrossmarkPage(Document):
doi = StringField(required=True)
is_doi_active = BooleanField(required=True, default=True)
enabled = BooleanField(required=True, default=True)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot default=False

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed to default=False in e8c9942.

Co-authored-by: robertatakenaka <505143+robertatakenaka@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adicione a CrossmarkPage o campo enabled como boolean

2 participants