Skip to content

Lift buttons - improved descriptions#750

Closed
trurlurl wants to merge 1 commit into
nextfrom
lift_buttons_doc
Closed

Lift buttons - improved descriptions#750
trurlurl wants to merge 1 commit into
nextfrom
lift_buttons_doc

Conversation

@trurlurl
Copy link
Copy Markdown
Contributor

@trurlurl trurlurl commented Jul 5, 2024

Disambiguating button descriptions. Problem description in https://3.basecamp.com/3256016/buckets/2570434/messages/7227314680

Please check whether my correction regarding centimeters -> metres is OK (LengthType should always be metres, in my view).

Disambiguating button descriptions.
@trurlurl trurlurl added bug Technical mistake, inconsistency with the documentation, etc. enhancement non semantic enhacement: technical enhancement, etc. needs documentation update The NeTEx document needs to be updated labels Jul 5, 2024
@trurlurl trurlurl added this to the netex_2.0 milestone Jul 5, 2024
@trurlurl trurlurl requested review from Aurige, skinkie and ue71603 July 5, 2024 19:24
@skinkie
Copy link
Copy Markdown
Contributor

skinkie commented Jul 5, 2024

Should these be propagated to master?

@skinkie skinkie changed the base branch from next to master July 9, 2024 07:43
@skinkie skinkie changed the base branch from master to next July 9, 2024 07:44
@skinkie
Copy link
Copy Markdown
Contributor

skinkie commented Jul 9, 2024

I'll close this pull request. It is a bug fix and should have targeted master.

@skinkie skinkie closed this Jul 9, 2024
@trurlurl trurlurl removed the needs documentation update The NeTEx document needs to be updated label Jul 23, 2024
@trurlurl
Copy link
Copy Markdown
Contributor Author

Removed "needs documentation updated" since closed without merging.

@skinkie skinkie deleted the lift_buttons_doc branch November 19, 2024 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Technical mistake, inconsistency with the documentation, etc. enhancement non semantic enhacement: technical enhancement, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants