Ability to testable SS API endpoint #30
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
Loader set Endpoint URL from Settings to Api client
|
Nestalo by zato, udelat tu GET cast zapamatovatelnejsi a trochu nadcasovou?
Napriklad ss_admin nebo ss_debug? Ci verzi s pomlckami?
Kdyz nam pribudou dalsi skryte fce, tak nebudeme muset vymyslet nic noveho.
(odeslano z mobilu)
Dne st 13. lis 2019 17:03 uživatel Jakub Bouček <[email protected]>
napsal:
… Občas potřebujeme testovat Wordpress plugin proti jinému serveru než
produkci a potřebujeme tedy změnit URL na testovací API endpoint. Zkusil
jsem proto upravit plugin, aby bylo možné nastavit vlastní URL API
endpointu.
Ale aby to nemátlo uživatele, je nastavení URL schováno tak a zobrazí se
jen při zadání parametru edit_url_endpoint do URL, např.:
/wp-admin/admin.php?page=ssc_options&edit_url_endpoint=yes
Nevím, zda je to ideální, popř. zda k tomu WP neposkytuje nějaké lepší
metody, kde číst parametry z URL. Pokud existují, tak mi to klidně omlaťte
o hlavu, od toho to PR děláme :)
Pokud by z kódu nebyl zřejmý záměr, podívejte se na jednotlivé commity, ty
jsou mnohem menší a srozumitelnější.
------------------------------
You can view, comment on, or merge this pull request online at:
#30
Commit Summary
- Settings: Add hidden settings to Admin for SimpleShop API Endpoint
URL
- Add unified loader for SimpleShop API client
- Typo: Rename method to more readable name
File Changes
- *M* src/Admin.php
<https://github.com/redbitcz/simpleshop-wp-plugin/pull/30/files#diff-0>
(2)
- *M* src/Plugin.php
<https://github.com/redbitcz/simpleshop-wp-plugin/pull/30/files#diff-1>
(22)
- *M* src/Settings.php
<https://github.com/redbitcz/simpleshop-wp-plugin/pull/30/files#diff-2>
(44)
Patch Links:
- https://github.com/redbitcz/simpleshop-wp-plugin/pull/30.patch
- https://github.com/redbitcz/simpleshop-wp-plugin/pull/30.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#30?email_source=notifications&email_token=AA7DI32BMVO2PC2LA66YLSTQTQQLLA5CNFSM4JM5SZMKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HZB5T4A>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA7DI35IMJSKJ3CF2RMMEDLQTQQLLANCNFSM4JM5SZMA>
.
|
|
@Dosty85 Hm, pravda, on takový Chceš si zkusit Git a udělat PR na PR? (vsichni víme, odkud to bylo odesláno…) |
Better $_GET param name for debug
Dosty85
approved these changes
Nov 14, 2019
jakubboucek
requested review from
vaclavgreif
and removed request for
vaclavgreif
vaclavgreif
reviewed
Nov 22, 2019
| 'desc' => __( '[SERVICE FLAG] You can here override URL to SimpleShop API. Leave blank to use default API.', 'simpleshop-cz' ), | ||
| 'id' => 'ssc_api_endpoint_url', | ||
| 'type' => 'text', | ||
| 'classes_cb' => [ $this, 'show_endpoint_url_when_flagged' ], |
There was a problem hiding this comment.
OK, založil jsem na to #33, ale v tomto PR to nechám, aby tomu byla zachována konzistence se zbytkem dosavadního kódu v pluginu.
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.
Občas potřebujeme testovat Wordpress plugin proti jinému serveru než produkci a potřebujeme tedy změnit URL na testovací API endpoint. Zkusil jsem proto upravit plugin, aby bylo možné nastavit vlastní URL API endpointu.
Ale aby to nemátlo uživatele, je nastavení URL schováno tak a zobrazí se jen při zadání parametru
edit_url_endpointdo URL, např.:/wp-admin/admin.php?page=ssc_options&edit_url_endpoint=yesNevím, zda je to ideální, popř. zda k tomu WP neposkytuje nějaké lepší metody, kde číst parametry z URL. Pokud existují, tak mi to klidně omlaťte o hlavu, od toho to PR děláme :)
Pokud by z kódu nebyl zřejmý záměr, podívejte se na jednotlivé commity, ty jsou mnohem menší a srozumitelnější.