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

Feat bgdidic 2841 tektonische karte #622

Merged
merged 3 commits into from
Jun 24, 2024

Conversation

faselm
Copy link
Contributor

@faselm faselm commented Jun 21, 2024

No description provided.

@github-actions github-actions bot added this to the 2024-07-03 milestone Jun 21, 2024
@faselm faselm requested review from marionb, rebert and ltclm June 21, 2024 13:12
Copy link
Contributor

@ltclm ltclm left a comment

Choose a reason for hiding this comment

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

for this to work as expected, you have to make sure that the id and the feature_id of the search result is unique on both tables. This is probably not the case with bgdi_id....
maybe you can use a union query like ch_bav_schienennetz or create a shared / unique sequence on both table, its name has to be different from bgdi_id ...

@faselm
Copy link
Contributor Author

faselm commented Jun 21, 2024

Thanks, does it work if I generate unique bgdi_id from FME?

@ltclm
Copy link
Contributor

ltclm commented Jun 21, 2024

yes that should work. Please make sure that

  • the column name is bgdi_id
  • there is a sequence on this column with the name %bgdi_id%

otherwise this cronjob here https://github.com/geoadmin/bgdi-scripts/blob/add0176a93445058b3d270557596430c1d409357/data-integration/automated/cron/includes/add_bgdi_id_column.sql
will change the column and its content and add a sequence. This is done every 15m.

@faselm
Copy link
Contributor Author

faselm commented Jun 21, 2024

Ok, I will just add a fid column it will be simpler

@faselm faselm requested a review from ltclm June 24, 2024 05:44
@faselm
Copy link
Contributor Author

faselm commented Jun 24, 2024

Hi @ltclm, the id field name was updated

Copy link
Contributor

@ltclm ltclm left a comment

Choose a reason for hiding this comment

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

thank you 👍

@faselm faselm merged commit 86bfd58 into develop-2024-07-03 Jun 24, 2024
5 checks passed
@faselm faselm deleted the feat-BGDIDIC-2841-tektonische_karte branch June 24, 2024 06:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants