Skip to content

feat: add explicit support for systemInfo and serverVersion config props#652

Merged
amcgee merged 3 commits into
masterfrom
feat/config-server-version
Oct 12, 2020
Merged

feat: add explicit support for systemInfo and serverVersion config props#652
amcgee merged 3 commits into
masterfrom
feat/config-server-version

Conversation

@amcgee
Copy link
Copy Markdown
Contributor

@amcgee amcgee commented Oct 9, 2020

This adds typescript annotations for systemInfo and serverVersion properties on the Config context object.

Note that these are currently added as Optional properties to avoid causing a breaking change, but eventually we will want them to be required.

@Mohammer5
Copy link
Copy Markdown
Contributor

Mohammer5 commented Oct 12, 2020

This is going to be used in the config service only, correct?
It'll be set by the app platform once dhis2/app-platform#470 has been merged, also correct?

EDIT: Just answered my questions by reading through the description of the app platform PR

@amcgee amcgee merged commit 15b8344 into master Oct 12, 2020
@amcgee amcgee deleted the feat/config-server-version branch October 12, 2020 08:52
dhis2-bot added a commit that referenced this pull request Oct 12, 2020
# [2.4.0](v2.3.0...v2.4.0) (2020-10-12)

### Features

* add explicit support for systemInfo and serverVersion config props ([#652](#652)) ([15b8344](15b8344))
@dhis2-bot
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 2.4.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants