add JetBrain Open Source Community Support #1442
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
qwerty287
approved these changes
Aug 5, 2022
nagmat84
approved these changes
Aug 5, 2022
Co-authored-by: Matthias Nagel <matthias.h.nagel@posteo.de>
d7415
reviewed
Aug 5, 2022
| @@ -85,6 +85,12 @@ In order to use the new photos email notification you will need to have configur | |||
|
|
|||
| Take a look at the [Documentation](https://lycheeorg.github.io/docs/), particularly the [FAQ](https://lycheeorg.github.io/docs/faq.html) if you have problems. Discovered a bug? Please create an issue [here](https://github.com/LycheeOrg/Lychee/issues) on GitHub! You can also contact us directly on [gitter (login with your github account). »](https://gitter.im/LycheeOrg/Lobby) | |||
|
|
|||
| ## Open Source Community Support | |||
|
|
|||
| <img src="https://resources.jetbrains.com/storage/products/company/brand/logos/PhpStorm_icon.png" alt="PhpStorm" width="50"/> | |||
There was a problem hiding this comment.
This upsets me on several levels. Mostly downloading a 200kb png for a 50px logo, which looks fairly clunky anyway.
There was a problem hiding this comment.
Agreed. :( That's their logo though and they don't provide lower res (but do provide svg). :(
Co-authored-by: Martin Stone <1611702+d7415@users.noreply.github.com>
ildyria
added a commit
that referenced
this pull request
Aug 6, 2022
* add JetBrain Open Source Community Support (#1442) * add JetBrain Open Source Community Support Co-authored-by: Matthias Nagel <matthias.h.nagel@posteo.de> Co-authored-by: Martin Stone <1611702+d7415@users.noreply.github.com> * Make VideoHandler support optional (#1439) * Make VideoHandler support optional * Changes recommended by nagmat84 * Add a testcase for video upload without ffmpeg * Fix new testcase * Fix a typo * Update tests/Feature/PhotosAddHandlerTestAbstract.php Co-authored-by: Matthias Nagel <matthias.h.nagel@posteo.de> * Check for error message that FFmpeg is disabled * Fixed Google Motion Photo test Co-authored-by: Matthias Nagel <matthias.h.nagel@posteo.de> * Apply fixes from PHP-CS-Fixer Co-authored-by: Benoît Viguier <ildyria@users.noreply.github.com> Co-authored-by: Matthias Nagel <matthias.h.nagel@posteo.de> Co-authored-by: Martin Stone <1611702+d7415@users.noreply.github.com> Co-authored-by: Kamil Iskra <kamil.01482@iskra.name> Co-authored-by: qwerty287 <80460567+qwerty287@users.noreply.github.com> Co-authored-by: github-actions[bot] <action@github.com>
nagmat84
added a commit
that referenced
this pull request
Sep 17, 2022
* Support API tokens per user * Do not require Accept and Content-Type headers * Fix 500 on user creation * Recompile * Fix some reviews * Undo Content-Type and Accept change * Fix phpstan * Fix phpstan * Fix phpstan 3 * add support for disabling the token completely rather than letting it set as a random value * disable-enable FK * add tests * remove unused variable * Translate and add strings * Fix security issue during migration that gives admin perms to api_key * Fix locale * Sync frontend * Suggestions * Apply suggestions from code review Co-authored-by: Matthias Nagel <matthias.h.nagel@posteo.de> * Apply suggestions * Recompile frontend * Fix perms * Fix lint * Improve check * Simplify middleware * Assert JSON * fix format * Apply fixes from PHP-CS-Fixer (#1444) * add JetBrain Open Source Community Support (#1442) * add JetBrain Open Source Community Support Co-authored-by: Matthias Nagel <matthias.h.nagel@posteo.de> Co-authored-by: Martin Stone <1611702+d7415@users.noreply.github.com> * Make VideoHandler support optional (#1439) * Make VideoHandler support optional * Changes recommended by nagmat84 * Add a testcase for video upload without ffmpeg * Fix new testcase * Fix a typo * Update tests/Feature/PhotosAddHandlerTestAbstract.php Co-authored-by: Matthias Nagel <matthias.h.nagel@posteo.de> * Check for error message that FFmpeg is disabled * Fixed Google Motion Photo test Co-authored-by: Matthias Nagel <matthias.h.nagel@posteo.de> * Apply fixes from PHP-CS-Fixer Co-authored-by: Benoît Viguier <ildyria@users.noreply.github.com> Co-authored-by: Matthias Nagel <matthias.h.nagel@posteo.de> Co-authored-by: Martin Stone <1611702+d7415@users.noreply.github.com> Co-authored-by: Kamil Iskra <kamil.01482@iskra.name> Co-authored-by: qwerty287 <80460567+qwerty287@users.noreply.github.com> Co-authored-by: github-actions[bot] <action@github.com> * Hash tokens * Sync frontend * Tmp commit to find out why it's failing * Second try to find failure * I'm pretty stupid at all * Fix db errors * Fix phpstan * Fix phpstan 2 * Fix phpstan 3 * Fix test * Add tests and fix issues as guest * sync frontend * fix phpdoc * Update locales and sync frontedn * Fix duplicated keys * Sync frontend * Add getAuthenticatedUser test * Don't send token with user, fix locales, synced frontend * Make PHPStan happy about something toally unrelated * Sync frontend * fix sync * Remove HasUserMiddleware * Fix tests * Added custom guard to handle session or token * Avoid unintended login as admin * Keep token-based authentication stateless + Fix tests * Added test for logout * Don't drop table * Recompile frontend * Revert "Recompile frontend" This reverts commit 039d72f. * Only migrate if not exists * Sync frontend to master * Undo change Co-authored-by: ildyria <beviguier@gmail.com> Co-authored-by: Matthias Nagel <matthias.h.nagel@posteo.de> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Benoît Viguier <ildyria@users.noreply.github.com> Co-authored-by: Martin Stone <1611702+d7415@users.noreply.github.com> Co-authored-by: Kamil Iskra <kamil.01482@iskra.name> Co-authored-by: github-actions[bot] <action@github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.