Skip to content

Project track configuration#60

Merged
martinyde merged 7 commits intodevelopfrom
feature/3071-dimensions
Nov 28, 2024
Merged

Project track configuration#60
martinyde merged 7 commits intodevelopfrom
feature/3071-dimensions

Conversation

@rimi-itk
Copy link
Copy Markdown
Contributor

@rimi-itk rimi-itk commented Nov 25, 2024

Link to ticket

https://leantime.itkdev.dk/TimeTable/TimeTable?showTicketModal=3071#/tickets/showTicket/3071

Description

  • Adds configuration to project track type (taxonomy term) and requires a dimensions property
  • Updates radios webform element to require options values for each defined dimension
  • Adds tests
  • @todo Use dimensions in calculations

Screenshot of the result

Defining dimensions:

Screen Shot 2024-11-27 at 09 44 46

Using dimensions:

Screen Shot 2024-11-27 at 09 45 16

Checklist

  • My code is covered by test cases.
  • My code passes our test (all our tests).
  • My code passes our static analysis suite.
  • My code passes our continuous integration process.

If your code does not pass all the requirements on the checklist you have to add a comment explaining why this change
should be exempt from the list.

Additional comments or questions

If you have any further comments or questions for the reviewer please add them here.

@rimi-itk rimi-itk force-pushed the feature/3071-dimensions branch 3 times, most recently from eff7f1b to 8279fcb Compare November 26, 2024 10:00
@rimi-itk rimi-itk force-pushed the feature/3071-dimensions branch from a2521e0 to e588c17 Compare November 26, 2024 10:16
@rimi-itk rimi-itk force-pushed the feature/3071-dimensions branch from e588c17 to 6b5f49f Compare November 26, 2024 10:43
@rimi-itk rimi-itk marked this pull request as ready for review November 27, 2024 08:43
@rimi-itk rimi-itk requested a review from martinyde November 27, 2024 08:48
@martinyde martinyde merged commit 2c253d0 into develop Nov 28, 2024
@martinyde martinyde deleted the feature/3071-dimensions branch November 28, 2024 07:19
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