Skip to content

Fix frontend-core.init #139

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

Merged
merged 4 commits into from
May 27, 2021
Merged

Fix frontend-core.init #139

merged 4 commits into from
May 27, 2021

Conversation

HustonMmmavr
Copy link
Contributor

No description provided.

* Cleanup `index_body` cache in case of re-init
* Add additional validation
* Prettified error messages
@HustonMmmavr HustonMmmavr requested a review from rosik May 26, 2021 16:07
@@ -5,6 +5,10 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
## [Unreleased]

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

CHANGELOG.md Outdated
Comment on lines 8 to 11
- Fixed `frontend-core.init`
- Cleanup `index_body` cache in case of re-init
- Add additional validation
- Prettified error messages
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Fixed `frontend-core.init`
- Cleanup `index_body` cache in case of re-init
- Add additional validation
- Prettified error messages
- Invalidate `index_body` cache in case of re-init.
- Make error messages more clear.
- Enhance options validation.

Copy link
Contributor

@rosik rosik left a comment

Choose a reason for hiding this comment

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

See two comments above. Apart from that LGTM.

@HustonMmmavr HustonMmmavr changed the title Fix frontend-core.init Fix frontend-core.init May 27, 2021
@HustonMmmavr HustonMmmavr merged commit 0fcfdd5 into master May 27, 2021
@HustonMmmavr HustonMmmavr deleted the fix-frontend-init branch May 27, 2021 09:59
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 this pull request may close these issues.

2 participants