Skip to content

[TASK] add attribution of device-pictures to about page#48

Merged
MyIgel merged 1 commit intofreifunk:mainfrom
maurerle:attribution
Oct 20, 2023
Merged

[TASK] add attribution of device-pictures to about page#48
MyIgel merged 1 commit intofreifunk:mainfrom
maurerle:attribution

Conversation

@maurerle
Copy link
Member

@maurerle maurerle commented Oct 1, 2023

Description

This adds attribution of the hardware images to the about page.
A rename from hwImg to device_pictures is done.

It is possible to change to a custom attribution if a different source of pictures is taken.

Motivation and Context

The CC-BY-NC-SA 4.0 requires to attribute the creators.
As we are all not lawyers this is a first step into this direction.
I do not think that it is required to attribute per picture, but we should make such information available in the https://github.com/freifunk/device-pictures repository.

How Has This Been Tested?

See https://map.aachen.freifunk.net/

Screenshots/links:

image

Checklist:

  • My code follows the code style of this project. (CI will test it anyway and also needs approval)
  • My change requires a change to the documentation.
    • I have updated the documentation accordingly.

@skorpy2009
Copy link
Member

@maurerle please rebase

Text is translatable in different languages with configurable source and license
@maurerle
Copy link
Member Author

maurerle commented Oct 11, 2023

Thanks, I rebased and added the suggestions from @MyIgel - and did run npm run lint:fix - really cool!

And I tested this for the FFAC config, which works fine

@maurerle
Copy link
Member Author

@MyIgel is there anything still needed to do here?

@MyIgel
Copy link
Member

MyIgel commented Oct 20, 2023

I don't think so, just had not enough time to have a last look ^^

@MyIgel MyIgel merged commit 35a9eef into freifunk:main Oct 20, 2023
@maurerle maurerle deleted the attribution branch December 30, 2025 17:56
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