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

Copy stop preparations #966

Merged
merged 5 commits into from
Dec 18, 2024
Merged

Copy stop preparations #966

merged 5 commits into from
Dec 18, 2024

Conversation

Huulivoide
Copy link
Contributor

@Huulivoide Huulivoide commented Dec 16, 2024

Add .idea to gitignore

Add extra fields needed in copies to StopPlace details query

Improve SimpleDropDownMenu

Improve LocatorButton

Implement placeholder action buttons on StopDetails page

  • Placeholder button for: Edit Stop validity period and/or priority.
  • Proper: OpenOnMap -button.
  • ExtraActions -menu with:
    • Placeholder meny items for: Make a new copy of stop -menu action.

This change is Reviewable

@Huulivoide Huulivoide force-pushed the CopyStopPreparations branch 2 times, most recently from 73ff3c4 to e8b9875 Compare December 17, 2024 08:25
* Placeholder button for: Edit Stop validity period and/or priority.
* Proper: OpenOnMap -button.
* ExtraActions -menu with:
  - Placeholder meny items for: Make a new copy of stop -menu action.
Copy link
Contributor

@suvikankare suvikankare left a comment

Choose a reason for hiding this comment

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

Reviewed 14 of 14 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @Huulivoide)

@Huulivoide Huulivoide merged commit 6f037d1 into main Dec 18, 2024
10 checks passed
@Huulivoide Huulivoide deleted the CopyStopPreparations branch December 18, 2024 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants