Bug: publication_approved never accesses errors param
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 25/100
Direção de pesquisa
Comece localizando o ponto de entrada lambda_handler e o método publication_approved; em seguida, rastreie como o valor errors é passado entre eles. A issue não informa o que publication_approved deve fazer com errors, portanto confirme o comportamento pretendido antes de alterar o código; a tarefa estará concluída quando o parâmetro tiver um uso definido e testado ou for removido de forma consistente.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Expected Behaviour
publication_approved method never access errors
Current Behaviour
No issue, parameter is never used
Code snippet
def lambda_handler(event, context):
"""Main entry point for Property Approval lambda function
Parameters
----------
event : API Gateway Lambda Proxy Request
The event passed to the function.
context : AWS Lambda Context
The context for the Lambda function.
Returns
-------
Success message upon successful storage of the approval outcome into DynamoDB
"""
logger.info(event)
errors = None if 'workflowErrors' not in event['detail'] else event['detail'].pop('workflowErrors')
# Deserialize event into strongly typed object
awsEvent:AWSEvent = Marshaller.unmarshall(event, AWSEvent) # type: ignore
detail:PublicationEvaluationCompleted = awsEvent.detail # type: ignore
return publication_approved(detail, errors)def lambda_handler(event, context):
"""Main entry point for Property Approval lambda function
Parameters
----------
event : API Gateway Lambda Proxy Request
The event passed to the function.
context : AWS Lambda Context
The context for the Lambda function.
Returns
-------
Success message upon successful storage of the approval outcome into DynamoDB
"""
logger.info(event)
errors = None if 'workflowErrors' not in event['detail'] else event['detail'].pop('workflowErrors')
# Deserialize event into strongly typed object
awsEvent:AWSEvent = Marshaller.unmarshall(event, AWSEvent) # type: ignore
detail:PublicationEvaluationCompleted = awsEvent.detail # type: ignore
return publication_approved(detail, errors)
Possible Solution
No response
Steps to Reproduce
N/A
Debugging logs
No response
- Linguagem predominante
- Python
- Estrelas
- 47
- Forks
- 20
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de aws-samples/aws-serverless-developer-experience-workshop-python
-
pending-release
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 35/100
aws-samples/aws-serverless-developer-experience-workshop-python#178 ·
-
Maintenance: function and test files do not match other runtimesTalvez livre de novo @sliedig assumiu há 1214 dias e não há nenhum pull request aberto. Abertainternal pending-release triage
aws-samples/aws-serverless-developer-experience-workshop-python#32 · 1 responsável ·
Todas as issues de aws-samples/aws-serverless-developer-experience-workshop-python
Issues semelhantes
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
awslabs/visual-asset-management-system#414 ·
Mantenedores costumam responder em até 1 dia
-
bug v1 v2
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
modelcontextprotocol/python-sdk#3670 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
aicell-lab/bioengine#232 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
modelscope/evalscope#1836 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 62/100