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

API Proposal for DynamicPeopleDensityInformation #335

Merged
merged 6 commits into from
Nov 30, 2023

Conversation

jgarciahospital
Copy link
Contributor

@jgarciahospital jgarciahospital commented Oct 28, 2023

@jgarciahospital jgarciahospital marked this pull request as ready for review October 28, 2023 10:58
Copy link
Contributor

@jordonezlucena jordonezlucena left a comment

Choose a reason for hiding this comment

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

Please, remove the file 'APIBacklog.md' from the PR, since it will be cause conflict with #331.
Leave only the md file with the API proposal.

@bigludo7
Copy link
Contributor

bigludo7 commented Nov 5, 2023

Hello @jgarciahospital - Is it same than #327 ?

@jordonezlucena
Copy link
Contributor

Hello @jgarciahospital - Is it same than #327 ?

They are related, but not the same - see preliminary discussion in the draft minutes of last conf. call (#338). Nonetheless, both APIs will be discussed together in next conf. call.

@bigludo7
Copy link
Contributor

bigludo7 commented Nov 6, 2023

Thanks @jordonezlucena for the clarification.

@jgarciahospital jgarciahospital marked this pull request as ready for review November 23, 2023 13:46
@jordonezlucena
Copy link
Contributor

The API proposal has been agreed to be taken for approval to TSC (7th Dec), so the PR is merged.

@jordonezlucena jordonezlucena merged commit 7a686eb into main Nov 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants