Skip to content

Code Quality: Use image metadata for address#18634

Open
yair100 wants to merge 1 commit into
mainfrom
ya/Images
Open

Code Quality: Use image metadata for address#18634
yair100 wants to merge 1 commit into
mainfrom
ya/Images

Conversation

@yair100

@yair100 yair100 commented Jun 23, 2026

Copy link
Copy Markdown
Member

Resolved / Related Issues

To prevent extra work, all changes to the Files codebase must link to an approved issue marked as Ready to build. Please insert the issue number following the hashtag with the issue number that this Pull Request resolves.

Steps used to test these changes

Stability is a top priority for Files and all changes are required to go through testing before being merged into the repo. Please include a list of steps that you used to test this PR.

  1. Confirmed image address meta data is correctly displayed

@yair100 yair100 assigned Josh65-2201 and unassigned Josh65-2201 Jun 23, 2026
@yair100 yair100 added the ready for review Pull requests that are ready for review label Jun 23, 2026
@yair100 yair100 requested a review from Josh65-2201 June 23, 2026 19:16
@Josh65-2201

Copy link
Copy Markdown
Member

Is the addressed suppose to be shown? PR left, 4.1.5 right

image

@mdtauk

This comment was marked as off-topic.

@Josh65-2201

This comment was marked as off-topic.

@yair100

yair100 commented Jun 24, 2026

Copy link
Copy Markdown
Member Author

Is the addressed suppose to be shown? PR left, 4.1.5 right

@Josh65-2201 I assumed this is what was requested in #10472. Is it supposed to work differently?

@Josh65-2201

Copy link
Copy Markdown
Member

In the description it mentions

Files should use it to build the address

In 4.1.5 it is, in this PR doesn't. That issue was made in v2.3 so it could have be added and not closed. The issue is quite wordy though

@yair100

yair100 commented Jun 24, 2026

Copy link
Copy Markdown
Member Author

My bad, I thought the screenshot on the right was this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature: Use IPTC Location metadata fields instead of reverse geocoding addresses

3 participants