Skip to content

[Bug] Fix outscrapper map review API along with moving to faster V3 API#250

Merged
lalitpagaria merged 4 commits intomasterfrom
fix_mpas_review
Jul 12, 2022
Merged

[Bug] Fix outscrapper map review API along with moving to faster V3 API#250
lalitpagaria merged 4 commits intomasterfrom
fix_mpas_review

Conversation

@lalitpagaria
Copy link
Copy Markdown
Collaborator

No description provided.

@lalitpagaria lalitpagaria added the bug Something isn't working label Jul 11, 2022
@lalitpagaria lalitpagaria requested a review from GirishPatel July 11, 2022 16:07
@lalitpagaria lalitpagaria self-assigned this Jul 11, 2022
@lalitpagaria lalitpagaria linked an issue Jul 11, 2022 that may be closed by this pull request
'sort': config.sort,
'start': since_timestamp,
'cutoff': config.until_timestamp,
'cutoff': since_timestamp,
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This doesn't look right.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

still working on it. Unable to test locally hence pushing to git and then test on colan

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Okay, cool !
I thought, it's open for review.

@lalitpagaria
Copy link
Copy Markdown
Collaborator Author

@GirishPatel Please review now

@lalitpagaria lalitpagaria merged commit fecb1c9 into master Jul 12, 2022
@lalitpagaria lalitpagaria deleted the fix_mpas_review branch July 12, 2022 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Map Review observer not honouring cutoff date

2 participants