Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Regroup internal EODAG params in a constants file #955

Open
bjgiraudon opened this issue Dec 6, 2023 · 0 comments
Open

Regroup internal EODAG params in a constants file #955

bjgiraudon opened this issue Dec 6, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@bjgiraudon
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
When developing on EODAG, parameters names, such as "missionStartDate" are usually hard-typed string, repeated in many places of the code base. Using a constant file would reduce the risk of typo happening when trying to access a given parameter, which value is constant.

Describe the solution you'd like
We could have something like an eodag internal constants file, listing the parameters that are used in EODAG and constant.
The code would import the constants from there.

@bjgiraudon bjgiraudon added the enhancement New feature or request label Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant