Skip to content
This repository has been archived by the owner on Oct 8, 2024. It is now read-only.

Fix/rulesheet expiration #99

Merged
merged 4 commits into from
Aug 7, 2023
Merged

Fix/rulesheet expiration #99

merged 4 commits into from
Aug 7, 2023

Conversation

ralphg6
Copy link
Member

@ralphg6 ralphg6 commented Aug 4, 2023

No description provided.

eliasfeijo
eliasfeijo reviewed Aug 4, 2023
View reviewed changes
Copy link
Contributor

@eliasfeijo eliasfeijo left a comment

Choose a reason for hiding this comment

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

Fiz apenas 2 comentários para você dar uma olhada, mas no geral está OK. Fico no aguardo do seu retorno para aprovar o PR.

services/rulles.go Outdated Show resolved Hide resolved
api.http Outdated Show resolved Hide resolved
eduardaguterres
eduardaguterres approved these changes Aug 7, 2023
View reviewed changes
Copy link
Contributor

@eduardaguterres eduardaguterres left a comment

Choose a reason for hiding this comment

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

LGTM

@eduardaguterres eduardaguterres merged commit 223b3f8 into develop Aug 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@eliasfeijo eliasfeijo eliasfeijo left review comments

@eduardaguterres eduardaguterres eduardaguterres approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ralphg6 @eliasfeijo @eduardaguterres