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

fix(export-dialog): was not closed when title was missing #1247

Merged
merged 1 commit into from
Feb 17, 2021

Conversation

JPSchellenberg
Copy link
Member

Hey,

this PR fixes a bug where the export dialog was not closeable anymore when exporting a file without giving a title.
Now the export dialog should also close on errors.

@JPSchellenberg JPSchellenberg self-assigned this Feb 16, 2021
@JPSchellenberg JPSchellenberg added the [type] bug Something isn't working (resulting in patch-version) label Feb 16, 2021
Copy link
Member

@sr258 sr258 left a comment

Choose a reason for hiding this comment

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

Looks alright

@JPSchellenberg JPSchellenberg merged commit 0c5f19c into master Feb 17, 2021
github-actions bot pushed a commit that referenced this pull request Feb 21, 2021
## [0.5.1](v0.5.0...v0.5.1) (2021-02-21)

### Bug Fixes

* **deps:** update dependency @lumieducation/h5p-express to v7.0.2 ([e6f5f8b](e6f5f8b))
* **deps:** update dependency @lumieducation/h5p-html-exporter to v7.0.2 ([75f9097](75f9097))
* **deps:** update dependency @lumieducation/h5p-server to v7.0.1 ([#1254](#1254)) ([eda9ef7](eda9ef7))
* **deps:** update dependency @lumieducation/h5p-server to v7.0.2 ([086e826](086e826))
* **deps:** update dependency @sentry/electron to v2.3.0 ([fc34fbc](fc34fbc))
* **deps:** update dependency @testing-library/react to v11.2.4 ([a43dafe](a43dafe))
* **deps:** update dependency @testing-library/react to v11.2.5 ([#1218](#1218)) ([6319384](6319384))
* **deps:** update dependency @testing-library/user-event to v12.6.3 ([39fa3d1](39fa3d1))
* **deps:** update dependency @testing-library/user-event to v12.7.0 ([54907fb](54907fb))
* **deps:** update dependency @testing-library/user-event to v12.7.1 ([4ea5292](4ea5292))
* **deps:** update dependency @types/node to v12.19.16 ([f5802f0](f5802f0))
* **deps:** update dependency @types/node to v12.20.0 ([a5eee8d](a5eee8d))
* **deps:** update dependency @types/node to v12.20.1 ([26cb252](26cb252))
* **deps:** update dependency @types/node to v12.20.2 ([9acf14e](9acf14e))
* **deps:** update dependency @types/react to v16.14.3 ([594edf0](594edf0))
* **deps:** update dependency @types/react to v16.14.4 ([aaebcf9](aaebcf9))
* **deps:** update dependency @types/react-dom to v16.9.11 ([0a77f5a](0a77f5a))
* **deps:** update dependency electron-log to v4.3.2 ([abc3967](abc3967))
* **deps:** update dependency i18next to v19.8.6 ([8ad28a7](8ad28a7))
* **deps:** update dependency i18next to v19.8.7 ([d95d3ef](d95d3ef))
* **deps:** update dependency i18next to v19.8.8 ([ccae144](ccae144))
* **deps:** update dependency i18next to v19.8.9 ([2d5a9eb](2d5a9eb))
* **deps:** update dependency react-scripts to v4.0.2 ([80ebfba](80ebfba))
* **deps:** update dependency typescript to v4.1.4 ([3dfdea4](3dfdea4))
* **deps:** update dependency typescript to v4.1.5 ([e91ebb8](e91ebb8))
* **deps:** update mui monorepo ([#1204](#1204)) ([35d3546](35d3546))
* **deps:** update sentry monorepo to v6 ([#1179](#1179)) ([8353eba](8353eba))
* **deps:** update sentry monorepo to v6.1.0 ([c51b761](c51b761))
* **deps:** update sentry monorepo to v6.2.0 ([166e3c8](166e3c8))
* **export-dialog:** was not closed when title was missing ([#1247](#1247)) ([0c5f19c](0c5f19c))
* **reporter:** fix big report files due to bg images ([#1229](#1229)) ([2abbc60](2abbc60)), closes [#1226](#1226)
@github-actions
Copy link

🎉 This PR is included in version 0.5.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@JPSchellenberg JPSchellenberg deleted the fix/export-dialog branch April 2, 2021 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released [type] bug Something isn't working (resulting in patch-version)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants