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

feat: hide button for unauthorized #1554

Closed
wants to merge 1 commit into from

Conversation

NtskwK
Copy link
Contributor

@NtskwK NtskwK commented Sep 6, 2024

@pierotofy
Copy link
Member

Did you test this?

@NtskwK
Copy link
Contributor Author

NtskwK commented Sep 7, 2024

Sorry, my problem. This is a very basic mistake.

@NtskwK
Copy link
Contributor Author

NtskwK commented Sep 7, 2024

@pierotofy Turn to #1555 , Please. Thank you very much.

@pierotofy
Copy link
Member

I'm confused, but this is still broken unfortunately.

@NtskwK I appreciate the help for the fixes you've been doing, but always, always test before opening a pull request. It's a lot of work for maintainers to review pull requests. Opening one and then asking for how to achieve a desired result is not the way to go.

If you have questions about development, we have a channel here: https://community.opendronemap.org/c/developers-chat/21

I'll close this for now, feel free to re-open once it's been tested and works? 🙏

@pierotofy pierotofy closed this Sep 7, 2024
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