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.

Sign up for GitHub

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

Jump to bottom

Update CHANGELOG.md #77

Merged
merged 2 commits into from
Mar 8, 2024
Merged

Update CHANGELOG.md #77

merged 2 commits into from
Mar 8, 2024

Conversation

ChetanT-System
Copy link
Contributor

@ChetanT-System ChetanT-System commented Mar 8, 2024 •
edited
Loading

  • Updated Known defects statements

Description

Known defects

  • Policy table misses details of the usage policies.
  • Data provider contracts overview should contain this information: kind of contracts, description, policies, information about the consumer.
  • Data consumer contracts overview should contain this information: kind of contracts, description, policies, information about the provider.

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

@ChetanT-System
Update CHANGELOG.md
ec6bc98 
- Updated Known defects statements
dvasunin
dvasunin approved these changes Mar 8, 2024
View reviewed changes
Copy link
Contributor

@dvasunin dvasunin left a comment

Choose a reason for hiding this comment

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

LGTM

sachinargade123
sachinargade123 approved these changes Mar 8, 2024
View reviewed changes
Copy link
Contributor

@sachinargade123 sachinargade123 left a comment

Choose a reason for hiding this comment

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

LGTM

@sachinargade123 sachinargade123 requested review from Ademandi and almadigabor and removed request for Ademandi March 8, 2024 08:42
@ChetanT-System
Update CHANGELOG.md
0e5604b 
- Removed non release points from change logs file.
@ChetanT-System
Copy link
Contributor Author

@almadigabor Please review and merge PR, PR have only changelog.md file changes.

@almadigabor almadigabor merged commit ab9022d into eclipse-tractusx:main Mar 8, 2024
4 checks passed
@almadigabor almadigabor deleted the Change-log-v24.03 branch March 8, 2024 09:40
Siegfriedk
Siegfriedk reviewed Mar 8, 2024
View reviewed changes
CHANGELOG.md
@@ -50,27 +49,6 @@ New features, fixed bugs, known defects and other noteworthy changes to each rel
### Changed
- Bumped version to 2.3.0 for helm charts to match with backend release.

## [2.1.4] - non-released
### Fixed
- Data table pagination not working - fixed
Copy link
Contributor

Choose a reason for hiding this comment

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

So you fixed it, removed it because you didn't do a release but never added it to another section? Not sure if i understnad your change

Copy link
Contributor

Choose a reason for hiding this comment

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

Hi @Siegfriedk , We have corrected it here #78

Thanks

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

@sachinargade123 sachinargade123 sachinargade123 approved these changes

@Siegfriedk Siegfriedk Siegfriedk left review comments

@jjeroch jjeroch jjeroch approved these changes

@almadigabor almadigabor almadigabor approved these changes

@dvasunin dvasunin dvasunin 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.
6 participants
@ChetanT-System @dvasunin @almadigabor @Siegfriedk @sachinargade123 @jjeroch