giandonatoinverso / PHP-Dynamic-Qr-code

PHP Dynamic Qr code is a script that allows the generation and saving of dynamic and static QR codes
MIT License
282 stars 79 forks source link

Embed Logo Inside QRCode #38

Open 0xRenegade opened 2 years ago

0xRenegade commented 2 years ago

Referencing issue which had a list of ideas, and since it was closed, sorting those into their own issues. (issue #27):

Basically the title, add option to embed logo inside a given qrcode. Food for thought: Would this still work on dynamic qrcodes?

giandonatoinverso commented 2 years ago

It could work but this project was born as a manager of dynamic qr code and should not deal with the actual generation.

In fact the qrcodes are generated by an external provider which, in theory, offers the possibility to insert a logo inside the qr code. However, it is known that adding images inside qrcode reduces its readability by the various readers in circulation. To customize the qrcode it would be better to work on the color or style of the same rather than trying to "embellish" it with graphic elements (eg logo images)

0xRenegade commented 2 years ago

It could work but this project was born as a manager of dynamic qr code and should not deal with the actual generation.

In fact the qrcodes are generated by an external provider which, in theory, offers the possibility to insert a logo inside the qr code. However, it is known that adding images inside qrcode reduces its readability by the various readers in circulation. To customize the qrcode it would be better to work on the color or style of the same rather than trying to "embellish" it with graphic elements (eg logo images)

Good points. Would it be better to make some kind of border/frame to go around it like a picture that has customized styling and/or images outside of the qrcode, such as a logo?

giandonatoinverso commented 2 years ago

I like. This is a good idea that is better suited to the concept of qr code manager and that frees customization from generation.

It would be necessary to intervene at the level of creation of the qr code, creating a new image based on the qr code generated by the provider and on the customization options previously chosen by the user.

0xRenegade commented 2 years ago

should we have the qrcode column like it is now, and add a new one for the new feature so that if they go to update it, it only changes the custom version, leaving the base in tact for use in updating in the future?

EDIT: This could also be a json encoded field for multiple custom versions?

giandonatoinverso commented 2 years ago

Yes, the imports must be incremental, i.e. the new qr codes not existing in the db must be added and those already existing but modified must be replaced

0xRenegade commented 2 years ago

This could also be a json encoded field for multiple custom versions, leaving it as a dynamic iteration of the custom versions they've created. or foreign key to new table (i like json for this one)

giandonatoinverso commented 2 years ago

Managing different versions of the same qrcode could be complex and not bring a concrete advantage to the user. It would also burden management and general usability.

It would be better to have, at least initially, a simple incremental qr code import system

RubenDebusscher commented 8 months ago

I was looking for a way to manage the QR's I will need, and to have some statistics about them, together with a lot of customisations. And I found this topic because I would like the option to add an image when needed. I started to look further, and I found this project EasyQRCodeJS. Maybe this might be a way to have a customisable self hosted generator inside the manager?

giandonatoinverso commented 8 months ago

Yes it could be a great generator. I invite you to contribute to the project

EduNetEurope commented 1 month ago

It would be absolutely amazing to add an image and have statistics. Any updates?

Baltimorepc commented 3 weeks ago

i see there's a plugin that should allow logos etc. Does anyone know how to add the plugin.

giandonatoinverso commented 3 weeks ago

Hi contact me on the e-mail present in the readme of this project.

Baltimorepc commented 1 week ago

i sent an email

giandonatoinverso commented 1 week ago

I replied but your mail server is not accepting replies

Baltimorepc commented 1 week ago

Someone had a crappy password and it was hacked before I updated the security policy so my server got on a black list strange that only some users have issues I’ll email u from my Gmail 

Sent from Yahoo Mail for iPhone

On Sunday, November 10, 2024, 4:20 PM, Giandonato Inverso @.***> wrote:

I replied but your mail server is not accepting replies

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>

Baltimorepc commented 1 week ago

thats very expensive. I dont need you to install it i already have i just wanted to add logos and images to the middle. ITs not worth all that. Ill look for another self hosted one thank you.

On Sun, Nov 10, 2024 at 4:51 PM chris adams @.***> wrote:

Someone had a crappy password and it was hacked before I updated the security policy so my server got on a black list strange that only some users have issues I’ll email u from my Gmail

Sent from Yahoo Mail for iPhone https://mail.onelink.me/107872968?pid=nativeplacement&c=Global_Acquisition_YMktg_315_Internal_EmailSignature&af_sub1=Acquisition&af_sub2=Global_YMktg&af_sub3=&af_sub4=100000604&af_sub5=EmailSignature__Static_

On Sunday, November 10, 2024, 4:20 PM, Giandonato Inverso < @.***> wrote:

I replied but your mail server is not accepting replies

— Reply to this email directly, view it on GitHub https://github.com/giandonatoinverso/PHP-Dynamic-Qr-code/issues/38#issuecomment-2466922163, or unsubscribe https://github.com/notifications/unsubscribe-auth/BB2JNZSH4GBHLXMWSI7MFDDZ77E3FAVCNFSM6AAAAABPTEJQIOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRWHEZDEMJWGM . You are receiving this because you commented.Message ID: @.***>