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

[feat] handshake fix & delete TempDir & dialog image error #20

Merged
merged 16 commits into from
Jan 9, 2022
Merged

[feat] handshake fix & delete TempDir & dialog image error #20

merged 16 commits into from
Jan 9, 2022

Conversation

philipkbh
Copy link
Member

@philipkbh philipkbh commented Jan 5, 2022 •
edited by joshuajeschek
Loading

closes #18

@philipkbh philipkbh linked an issue Jan 5, 2022 that may be closed by this pull request
Versuchsbild in installierter Toolbox wird nicht angezeigt [ZoomMaps] #18
Closed
@joshuajeschek
Copy link
Member

damit das issue geclosed wird, reicht es nicht es zu verlinken @TheDarkDesync
habe das benötigte zur beschreibung hinzugefügt

@joshuajeschek joshuajeschek added this to the toolbox v1.0 milestone Jan 6, 2022
@joshuajeschek
Copy link
Member

Wenn wir das jetzt schon mit einem temp ordner machen, können wir uns da noch drum kümmern dass er wieder gelöscht wird?
https://docs.oracle.com/javase/7/docs/api/java/lang/Runtime.html#addShutdownHook(java.lang.Thread)
wenn du willst kann ich das auch machen

@philipkbh
Copy link
Member Author

damit das issue geclosed wird, reicht es nicht es zu verlinken @TheDarkDesync habe das benötigte zur beschreibung hinzugefügt

alles klar

@philipkbh
Copy link
Member Author

Wenn wir das jetzt schon mit einem temp ordner machen, können wir uns da noch drum kümmern dass er wieder gelöscht wird? https://docs.oracle.com/javase/7/docs/api/java/lang/Runtime.html#addShutdownHook(java.lang.Thread) wenn du willst kann ich das auch machen

well vielleicht habe ich das überlesen xD ich probiere mich mal ran

@joshuajeschek
Copy link
Member

Das Bild muss aber übrigens im Versuch geladen werden, nicht beim Erstellen des Test Versuches :)
in der Konfiguration steht dann nur die URL

@philipkbh
Copy link
Member Author

Das Bild muss aber übrigens im Versuch geladen werden, nicht beim Erstellen des Test Versuches :) in der Konfiguration steht dann nur die URL

stimmt xD

@philipkbh philipkbh changed the title [fix] handshake problem fix [feat] handshake fix & delete TempDir & dialog image error Jan 8, 2022
joshuajeschek
joshuajeschek approved these changes Jan 8, 2022
View reviewed changes
Copy link
Member

@joshuajeschek joshuajeschek left a comment

Choose a reason for hiding this comment

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

jetzt siehts gut aus!

Rotwolf
Rotwolf approved these changes Jan 9, 2022
View reviewed changes
Copy link
Contributor

@Rotwolf Rotwolf left a comment

Choose a reason for hiding this comment

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

👍

@philipkbh philipkbh merged commit f289dd6 into main Jan 9, 2022
@philipkbh philipkbh deleted the handshake-fix branch January 9, 2022 16:29
joshuajeschek added a commit that referenced this pull request Jan 10, 2022
@philipkbh @joshuajeschek
[feat] handshake fix & delete TempDir & dialog image error (#20)
87adde9 
* [fix] handshake problem fix

* [feat] delete tmpdir on exit

* [test] Exceptions

* [res] added no-image-image

* [feat] dialog when image cant be resolved

* [refactor] deleteTempDir name change

* [feat] image for dialog

* [fix] removed saveImage

* [test] fixed exception

* [style] Formattierung und @throws clauses

* [feat] style enhancements

* [fix] informativere message, größeres Feld, Icon

* [fix] tmpdirprefix + Logger.info

Co-authored-by: Joshua Jeschek <64850647+joshuajeschek@users.noreply.github.com>
Co-authored-by: joshuajeschek <joshua.jeschek@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joshuajeschek joshuajeschek joshuajeschek approved these changes

@Rotwolf Rotwolf Rotwolf approved these changes

@Nilrei Nilrei Awaiting requested review from Nilrei

@sar-sturm sar-sturm Awaiting requested review from sar-sturm

@justin7en justin7en Awaiting requested review from justin7en

@grenoui11e grenoui11e Awaiting requested review from grenoui11e

Assignees

@joshuajeschek joshuajeschek

@philipkbh philipkbh

Labels
None yet
Projects
None yet
Milestone
toolbox v1.0
Development

Successfully merging this pull request may close these issues.

Versuchsbild in installierter Toolbox wird nicht angezeigt [ZoomMaps]
3 participants
@philipkbh @joshuajeschek @Rotwolf