NB-Public
/
WolKal3000
Archived
11
1
Fork 0
Code Issues Pull Requests Releases 2 Wiki Activity

vorkonfiguriert fuer die Wolke #13

Merged
christian.tramnitz merged 3 commits from feature/wolke-preconfig into main 2021-01-18 17:26:45 +01:00
Conversation 0 Commits 3 Files Changed 10 +473 -672
christian.tramnitz commented 2020-12-21 20:32:06 +01:00
Owner
Copy Link
  • redundante readme.txt entfernt
  • GCAL namespace auf wolkal3000 geändert (Dateinamen, WP options) (dadurch ggf. Co-Existenz mit dem ursüringlichen GCal-Plugin möglich!)
  • Vorkonfiguration für die Wolke, man braucht nur die 16-stellige Kalender ID (dadurch Security Verbesserung: kein dynamisches Laden von potentiell nicht vertrauenswürdigen Inhalten)
  • Google Maps Funktionen entfernt

Bei Migration von <= 0.3.8 bitte das alte Plugin per "uninstall" deinstallieren, damit die Daten im alten Namespace gelöscht werden.
Achtung: Die Konfiguration muss (im neuen Namespace) neu vorgenommen werden!

* redundante readme.txt entfernt * GCAL namespace auf wolkal3000 geändert (Dateinamen, WP options) (dadurch ggf. Co-Existenz mit dem ursüringlichen GCal-Plugin möglich!) * Vorkonfiguration für die Wolke, man braucht nur die 16-stellige Kalender ID (dadurch Security Verbesserung: kein dynamisches Laden von potentiell nicht vertrauenswürdigen Inhalten) * Google Maps Funktionen entfernt Bei Migration von <= 0.3.8 bitte das alte Plugin per "uninstall" deinstallieren, damit die Daten im alten Namespace gelöscht werden. Achtung: Die Konfiguration muss (im neuen Namespace) neu vorgenommen werden!
JunWi was assigned by christian.tramnitz 2020-12-21 20:32:06 +01:00
kim.lang was assigned by christian.tramnitz 2020-12-21 20:32:06 +01:00
sebastian.steinbach was assigned by christian.tramnitz 2020-12-21 20:32:06 +01:00
christian.tramnitz added 1 commit 2020-12-21 20:32:06 +01:00
willi.junga reviewed 2020-12-22 07:44:54 +01:00
README.md Outdated
@ -7,3 +7,3 @@
## Warnung
Nicht produktiv verwenden. Nur zu Testzwecken.
Das hier ist noch in einem frühen Entwicklungsstadium aber durch die Begrenzung auf eine (vertrauenswürdige!) Quelle, sollte sich möglicher Sachden in Grenzen halten.
willi.junga commented 2020-12-22 07:44:54 +01:00
Owner

Sachden > Schaden

Sachden > Schaden
willi.junga marked this conversation as resolved
christian.tramnitz added 1 commit 2020-12-22 10:19:40 +01:00
willi.junga approved these changes 2020-12-31 21:22:32 +01:00
willi.junga left a comment
Owner

Ich habe mir jetzt nur die änderungen angeguckt und nur das kommentiert von dem ich meine irgendwas so halbwegs verstanden zu haben oder vermute dass etwas anders funtkionieren könnte.

sonst sind die meisten änderungen ja nur namensänderungen was soweit für mich hier passt.

Ich habe mir jetzt nur die änderungen angeguckt und nur das kommentiert von dem ich meine irgendwas so halbwegs verstanden zu haben oder vermute dass etwas anders funtkionieren könnte. sonst sind die meisten änderungen ja nur namensänderungen was soweit für mich hier passt.
wolkal3000-admin.php
@ -0,0 +55,4 @@
add_action('admin_init', 'wolkal3000_admin_init');
function wolkal3000_admin_init(){
register_setting( 'wolkal3000_options', 'wolkal3000_options', 'wolkal3000_options_validate' );
willi.junga commented 2020-12-31 20:49:34 +01:00
Owner

Muss das doppelt sein? Ich hab keine Ahnung:

'wolkal3000_options', 'wolkal3000_options'
Muss das doppelt sein? Ich hab keine Ahnung: ``` 'wolkal3000_options', 'wolkal3000_options' ```
christian.tramnitz commented 2021-01-04 18:06:52 +01:00
Author
Owner

vorher:
function gcal_admin_init(){
register_setting( 'gcal_options', 'gcal_options', 'gcal_options_validate' );

Laut WP Doku (https://developer.wordpress.org/reference/functions/register_setting/) ist der erste Wert der Name optrion_group Name, der zweite Wert der option_name. Man braucht also auf jeden Fall zwei Paramater. Die müssen sicherlich nicht zwnagsläufig gleich heißen, aber ich wüsste auch nicht, warum man jetzt davon abweichen sollte, wenn es vorerh auch so war.

vorher: function gcal_admin_init(){ register_setting( 'gcal_options', 'gcal_options', 'gcal_options_validate' ); Laut WP Doku (https://developer.wordpress.org/reference/functions/register_setting/) ist der erste Wert der Name optrion_group Name, der zweite Wert der option_name. Man braucht also auf jeden Fall zwei Paramater. Die müssen sicherlich nicht zwnagsläufig gleich heißen, aber ich wüsste auch nicht, warum man jetzt davon abweichen sollte, wenn es vorerh auch so war.
JunWi marked this conversation as resolved
wolkal3000-admin.php
@ -0,0 +83,4 @@
function wolkal3000_feeds_section_text() {
?>
<p><b>Wolke-Kalender synchronisieren in eine ausgewählte Terminkategorie von Kal3000. Bitte trage hierfür die entsprechende Export-Adresse des gewünschten Wolke-Kalenders ein.</b></br><b>Falls zu einer Terminkategorie kein Wolke-Kalender synchronisiert werden soll, entsprechendes Feld bitte leer lassen.</b></p>
<p><b><a href="https://confluence.netzbegruenung.de/pages/viewpage.action?pageId=40730689" target="_blank">Erfahre mehr darüber, wie du die Export-Adresse eines Wolke-Kalenders findest.</a></b>
willi.junga commented 2020-12-31 20:52:19 +01:00
Owner

würde vorschlagen eine redirect URL zu verwenden: https://git.verdigado.com/Netzbegruenung/support-url-redirects

confluence hat ende februar glaube ich keine lizenz mehr :(

würde vorschlagen eine redirect URL zu verwenden: https://git.verdigado.com/Netzbegruenung/support-url-redirects confluence hat ende februar glaube ich keine lizenz mehr :(
christian.tramnitz commented 2021-01-04 18:11:26 +01:00
Author
Owner

Ist noch etwas länger als Februar, aber stimmt schon.
Netzbegruenung/support-url-redirects#4

Ist noch etwas länger als Februar, aber stimmt schon. https://git.verdigado.com/Netzbegruenung/support-url-redirects/pulls/4
willi.junga marked this conversation as resolved
wolkal3000-admin.php Outdated
@ -0,0 +92,4 @@
function wolkal3000_feeds_setting_string($args) {
$options = get_option('wolkal3000_options');
$placeholder = "SEZ1234567890ABC";
willi.junga commented 2020-12-31 20:59:53 +01:00
Owner

DEADBEEFDEADBEEF, wäre leichter als Platzhalter verständlich, aber deine entscheidung. (:

`DEADBEEFDEADBEEF`, wäre leichter als Platzhalter verständlich, aber deine entscheidung. (:
christian.tramnitz commented 2021-01-04 18:16:29 +01:00
Author
Owner

ein paar Kalender von mir fingen tatsächlich mit SEZ an, deswegen dachte ich das wäre ein gutes Beispiel... aber anscheinend nur Zufall. deadbeef ist eher in Techie-Kreisen bekannt, impliziert aber hexadezimalen Input, hier geht es um einen beliebigen 16stelligen ASCII string. Deswegen wäre deadbeef imho eher verwirrend, können aber auch gern was anderes als mein Beispiel nehmen.

ein paar Kalender von mir fingen tatsächlich mit SEZ an, deswegen dachte ich das wäre ein gutes Beispiel... aber anscheinend nur Zufall. deadbeef ist eher in Techie-Kreisen bekannt, impliziert aber hexadezimalen Input, hier geht es um einen beliebigen 16stelligen ASCII string. Deswegen wäre deadbeef imho eher verwirrend, können aber auch gern was anderes als mein Beispiel nehmen.
JunWi marked this conversation as resolved
wolkal3000-admin.php Outdated
@ -0,0 +111,4 @@
function wolkal3000_timer_setting_string() {
$options = get_option('wolkal3000_options');
$placeholder = "default 60";
echo '<input type="text" id="wolkal3000_timer" name="wolkal3000_options[wolkal3000_timer]" value="' . $options['wolkal3000_timer'] . '" size="6" maxlength="6" placeholder="' . $placeholder . '" > Minuten </br>';
willi.junga commented 2020-12-31 21:10:05 +01:00
Owner

koennten wir auf maxlength="4" stellen wäre bei 9999 dann etwa 1x pro woche.

koennten wir auf maxlength="4" stellen wäre bei 9999 dann etwa 1x pro woche.
christian.tramnitz commented 2021-01-04 18:19:18 +01:00
Author
Owner

jo, kein problem.
Ich warte noch Feedback zu den anderen Punkten ab bevor ich den PR aktualisiere.

jo, kein problem. Ich warte noch Feedback zu den anderen Punkten ab bevor ich den PR aktualisiere.
christian.tramnitz added 1 commit 2021-01-18 17:24:39 +01:00
christian.tramnitz merged commit 3f03ea7cda into main 2021-01-18 17:26:45 +01:00
This repo is archived. You cannot comment on pull requests.
Reviewers
No reviewers
willi.junga
Labels
Clear labels
  
Effort
High
estimated 24 hours of work

  
Effort
Low
estimated 4 hours of work

  
Effort
Medium
estimated 12 hours of work

  
Prio
High
Mission critical

  
Prio
Low
Nice to have

  
Prio
Medium
Improves function or usability substantially

  
Type
Bug
Prevents existing functionality from being used

  
Type
Feature
Adds new features

No Label
Effort
High
Effort
Low
Effort
Medium
Prio
High
Prio
Low
Prio
Medium
Type
Bug
Type
Feature
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
JunWi
kim.lang
sebastian.steinbach
2 Participants
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: NB-Public/WolKal3000#13
No description provided.
Delete Branch "feature/wolke-preconfig"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?