Skip to content
This repository has been archived by the owner on Jun 3, 2024. It is now read-only.

Change links to instead edit POI #321

Merged
merged 3 commits into from
Jul 30, 2019
Merged

Change links to instead edit POI #321

merged 3 commits into from
Jul 30, 2019

Conversation

GuillaumeGomez
Copy link
Contributor

@GuillaumeGomez GuillaumeGomez commented Jul 30, 2019 •
edited
Loading

Screenshot from 2019-07-30 16-18-19

bbecquet
bbecquet suggested changes Jul 30, 2019
View reviewed changes
Copy link
Contributor

@bbecquet bbecquet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you provide a screenshot in the PR description please?

bbecquet
bbecquet previously approved these changes Jul 30, 2019
View reviewed changes
amatissart
amatissart reviewed Jul 30, 2019
View reviewed changes
Copy link
Contributor

@amatissart amatissart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A minor wording suggestion

src/views/poi_panel.dot Outdated Show resolved Hide resolved
@GuillaumeGomez @amatissart
Update src/views/poi_panel.dot
9a8ca1a 
Co-Authored-By: Adrien Matissart <amatissart@users.noreply.github.com>
@GuillaumeGomez GuillaumeGomez dismissed stale reviews from bbecquet via 9a8ca1a July 30, 2019 14:52
@GuillaumeGomez GuillaumeGomez merged commit 143e035 into Qwant:master Jul 30, 2019
@GuillaumeGomez GuillaumeGomez deleted the two-links branch July 30, 2019 15:01
@amatissart amatissart mentioned this pull request Aug 16, 2019
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@amatissart amatissart amatissart approved these changes

@bbecquet bbecquet bbecquet left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@GuillaumeGomez @bbecquet @amatissart