Skip to content

Conversation

@printminion-co
Copy link
Contributor

@printminion-co printminion-co commented May 15, 2024

  • check if CSP header is properly set and delivered via browser

@thlehmann-ionos
Copy link
Collaborator

Intermediate feedback:

  • We agreed upon the naming scheme "nc-", please rename the repo accordingly (See Google group chat amongst Matthias, you and me, search for "Neue Repos werden mit "nc-" Prefixed")
  • Consider renaming the app to "googleanalytics" (since "analytics" is already an official name)
  • Please remove the PHP tests
  • Please remove the .gitattributes as it declares js/track.js as binary which hides changes in git log. There's also no reason for it.
  • What's the reason for the reuse: commit and added license files with copyright holder/date placeholders. Where is this from?

Review ongoing.

@thlehmann-ionos
Copy link
Collaborator

@thlehmann-ionos
Copy link
Collaborator

Review done for the code part, ongoing for manual tests.

@thlehmann-ionos
Copy link
Collaborator

Is the Makefile still relevant and uptodate? Please revise the instructions in the readme.

@printminion-co printminion-co force-pushed the mk/dev/HDFRONT-6777-enc_configure branch 10 times, most recently from d5cf50f to 585b5cb Compare May 29, 2024 16:33
@printminion-co
Copy link
Contributor Author

review amended

  • added rudimentary php tests
  • made PR green

@printminion-co printminion-co force-pushed the mk/dev/HDFRONT-6777-enc_configure branch 12 times, most recently from 6e46bfd to 55b6d37 Compare June 3, 2024 08:06
@printminion-co printminion-co force-pushed the mk/dev/HDFRONT-6777-enc_configure branch from 55b6d37 to 7882c51 Compare June 12, 2024 13:34
Signed-off-by: Misha M.-Kupriyanov <[email protected]>
Signed-off-by: Misha M.-Kupriyanov <[email protected]>
Signed-off-by: Misha M.-Kupriyanov <[email protected]>
in order to be compatible with future php releases

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
run: npm install
Signed-off-by: Misha M.-Kupriyanov <[email protected]>
Signed-off-by: Misha M.-Kupriyanov <[email protected]>
Signed-off-by: Misha M.-Kupriyanov <[email protected]>
in order to avoid exception

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
Signed-off-by: Misha M.-Kupriyanov <[email protected]>
Signed-off-by: Misha M.-Kupriyanov <[email protected]>
Signed-off-by: Misha M.-Kupriyanov <[email protected]>
in order to reflect what linter does

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
for security reasons

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
Signed-off-by: Misha M.-Kupriyanov <[email protected]>
Signed-off-by: Misha M.-Kupriyanov <[email protected]>
since we are programming for v29

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
in order to add additional values

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
Error:
Using default tag: latest
  latest: Pulling from deepdiver/docker-oracle-xe-11g
  [DEPRECATION NOTICE] Docker Image Format v1 and Docker Image manifest version 2, schema 1 support is disabled by default and will be removed in an upcoming release. Suggest the author of docker.io/deepdiver/docker-oracle-xe-11g:latest to upgrade the image to the OCI Format or Docker Image manifest v2, schema 2. More information at

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
@printminion-co printminion-co force-pushed the mk/dev/HDFRONT-6777-enc_configure branch from 5d0d430 to 7beba14 Compare July 11, 2024 09:23
@thlehmann-ionos
Copy link
Collaborator

Explain magic strings in b22d4cb

@thlehmann-ionos
Copy link
Collaborator

@thlehmann-ionos thlehmann-ionos merged commit 0aa10cf into main Jul 11, 2024
@thlehmann-ionos thlehmann-ionos deleted the mk/dev/HDFRONT-6777-enc_configure branch July 11, 2024 11:58
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.

3 participants