Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

iD Tagging scheme exploration #131

Merged
merged 32 commits into from
Jun 17, 2023
Merged

iD Tagging scheme exploration #131

merged 32 commits into from
Jun 17, 2023

Conversation

zbycz
Copy link
Owner

@zbycz zbycz commented Feb 2, 2023

An attemt to adopt the presets from id-tagging-schema project and show them in very simple way on FeaturePanel.

Follow-up:

TODO

  • types from json-schemas
  • some tests
  • find out what is missing from spec
  • implement all field types and some special like opening_hours
  • connect to lanuage switcher
  • refactor WIP
  • maybe write osmApi without need for intl? (+fix test)
  • get presets + fields from npm

Bugs:

@vercel
Copy link

vercel bot commented Feb 2, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
osmapp ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 17, 2023 2:34pm

Committed in-flight 5J814 :)
# Conflicts:
#	yarn.lock
@zbycz zbycz marked this pull request as ready for review June 17, 2023 14:34
@@ -1,8 +1,8 @@
import { Feature } from '../services/types';
import { roundedToDeg } from '../utils';

const getSubclass = ({ properties, osmMeta }: Feature) =>
properties.subclass?.replace(/_/g, ' ') || osmMeta.type; // TODO translate ? maybe use iD editor logic (already with translations)
Copy link
Owner Author

Choose a reason for hiding this comment

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

@zbycz zbycz merged commit b3c206b into master Jun 17, 2023
@zbycz zbycz deleted the tagging branch June 17, 2023 14:55
whubsch pushed a commit to whubsch/osmapp that referenced this pull request Jun 20, 2023
* iD Tagging scheme exploration

* fix build

* fix build2

* show only unique fields

* add typescript types

* y lintfix

* find Field for rest of tags

* move schema to Feature + add UI

* translate values in semiCombo

* titles

* hack address:* in

* remove tags which are already covered by Preset name

* experiment - clear sessionstorage

* typeCombo is usually cleared by preset.tags

* We need typeCombo after all, eg node/7002486683

* move units from label to value

* finalize first prototype

* allow language switching

* fix buildAddress a little

* extract PoiDescription.tsx

* extract field helpers to fields.ts

* refactoring restKeys->keysTodo, added publishDbgObject(), added test

* fix access key - covers multiple tags

* Use `@openstreetmap/id-tagging-schema` pkg

Committed in-flight 5J814 :)

* fix keysTodo (never commit 30k ft above ground)

* y lintfix

* fix [object object] in `subject:wikidata`

* update osmApi.test.ts

* hide under advanced mode

* add Details above TagsTable
zbycz added a commit to whubsch/osmapp that referenced this pull request Jun 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use iD presets
2 participants