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

Replace error window on first export #96

Merged

Conversation

patrickmollohan
Copy link
Contributor

This PR streamlines the process of the first export of a project. Rather than prompting the user to open the export settings via an error window, Diz will now do so automatically.

@binary1230 binary1230 merged commit 449d284 into IsoFrieze:master Dec 14, 2023
1 check passed
@binary1230
Copy link
Collaborator

Hey, you seem like you know what you're doing, so I'm just going to invite you to the repo and give write access. (tag... you're it)

Feel free to keep doing stuff like this. if it's something larger, just open a PR and tag me, I can have a look. for more tweak-y UI stuff, feel free to just merge directly in. we'll do that til things break : )

@binary1230
Copy link
Collaborator

er... oops apparently I don't actually have access to add people (it's still on @IsoFrieze's account). I'll get the rest of these PRs merged in the meantime.

@patrickmollohan
Copy link
Contributor Author

Haha well I'm honoured! To be honest, Diz is my first time fiddling around with C#, so I think you might be giving me a wee too much credit there. Nevertheless, I will continue finding meaningful ways to contribute in the mean time! :D

@patrickmollohan patrickmollohan deleted the qol/streamline-first-export branch December 15, 2023 19:25
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.

None yet

2 participants