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

add eco-responsible pannel for desktop #1416

Merged
merged 9 commits into from
Oct 27, 2022
Merged

Conversation

remi-dupre
Copy link
Contributor

@remi-dupre remi-dupre commented Oct 26, 2022

Add new buttons in the category panel for eco-responsible POIs.

Note that only the "public bookshelf" category is implemented yet in the API and that this needs TILEVIEW_features_ecoResponsible=1 in the env to be displayed.

desktop ............................................................................................................. mobile
Screenshot 2022-10-26 at 16-05-46 Qwant Maps - La cartographie qui ne vous trace pas Screen Shot 2022-10-26 at 16 05 09

@remi-dupre remi-dupre requested a review from sdrll October 26, 2022 14:10
@remi-dupre remi-dupre self-assigned this Oct 26, 2022
@remi-dupre remi-dupre force-pushed the eco-responsible-categories branch 2 times, most recently from ab02302 to 9f33f48 Compare October 26, 2022 14:15
@remi-dupre remi-dupre marked this pull request as ready for review October 26, 2022 14:15
sdrll
sdrll previously approved these changes Oct 26, 2022
name: supermarket
color: "#a125be"
color: '#a125be'
matcher:
regex: superettes?|epiciers?|epiceries?|8 [aà] huit|leader price express|coccimarket|naturalia|viveco|utile|proxi service|kusmi tea|votre marche|epi service|satoriz|proxi marche|spar supermarche|day by day|palais des thes|le monde du macaron|point coop|le marche d'a cote|corsaire|cali|g 20|paris store|hyper casino|les halles? de l'aveyron|rnpc|ecofrais|europrix|easy marche|ferrari|intermarche express|kelly services?|partisans? du gout|(?:super|hyper|inter)marches?|auchan|carrefour|(?:e\.)?leclerc|casino|(?:super|hyper)\s+u|monoprix|simply|u\s+express|proxi|spar|vival|lidl|leader\s+price|grand\s+frais|petit casino|leader price|picard|franprix|super u|simply market|casino shop|grand frais|netto|8 a huit|coop|monop'|proxi super|panier sympa|supermarche match|chronodrive|atac|colruyt|casino supermarche|coccinelle express|cora|sherpa|aldi|maximarche|bi1|marche u|shopi|ecomax|relais des mousquetaires|sitis|champion|intermarche contact|marche plus|score|a2pas|geant casino|norma|systeme u|jumbo score|o'tera|coccinelle supermarche|ecomarche|espace multimedia|express|intermarche super|casino drive|costco wholesale|franprix nano|galeries gourmandes|hyper champion|partisans du gout|simply city|u drive|unik market|carrefour market|intermarche
Copy link
Contributor

@sdrll sdrll Oct 26, 2022

Choose a reason for hiding this comment

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

I am not sure these regex are used anymore, it could be move in the back end

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍

</Panel>
{ecoResponsible && (
<>
Copy link
Contributor

@sdrll sdrll Oct 26, 2022

Choose a reason for hiding this comment

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

is it on purpose <> ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh you're right, I'm not sure how it slipped here as there is only one child component, Panel, it may just be due to an older attempt at implementing this 🤷

@remi-dupre remi-dupre merged commit 6b9ab0a into master Oct 27, 2022
@remi-dupre remi-dupre deleted the eco-responsible-categories branch October 27, 2022 08:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants