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

Reverse logic for missing uid #45

Closed
TeodorTunev opened this issue Jan 17, 2022 · 4 comments
Closed

Reverse logic for missing uid #45

TeodorTunev opened this issue Jan 17, 2022 · 4 comments

Comments

@TeodorTunev
Copy link

Hello, we have notice that SDK 1.8.0 generate a lot of error for Entry Uid is required, after debugging it we found out that the logic for empty uid is wrong here - https://github.com/contentstack/contentstack-java/blob/master/src/main/java/com/contentstack/sdk/Entry.java#L955 , its check if its not empty, then throw an error. Can we release minor version with the fix please, we need 1.8.0, because of log4j update, but our logs are full of that error.

@ishaileshmishra
Copy link
Member

Thanks for writing to us @TeodorTunev. We have noticed the issue. We will fix it earliest and update you.

ishaileshmishra added a commit that referenced this issue Jan 22, 2022
- Reverse logic for entry uid
- Static logger
- scope based dependencies
@TeodorTunev
Copy link
Author

TeodorTunev commented Jan 24, 2022

Hello @ishaileshmishra, I see that you fix the bug, when you plan to release the version?

@ishaileshmishra
Copy link
Member

@TeodorTunev We are scheduling to release it by end of the month.

ishaileshmishra added a commit that referenced this issue Jan 27, 2022
ishaileshmishra added a commit that referenced this issue Jan 27, 2022
- Reverse logic for entry uid
- Static logger
- scope based dependencies
ishaileshmishra added a commit that referenced this issue Jan 27, 2022
@ishaileshmishra
Copy link
Member

The issue has been fixed in v1.8.1, Please to the latest version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants