Remove ProjectIDEvent
- Dominant language
- C#
- Stars
- 9
- Forks
- 6
- PR merge metrics
- No merged PRs in 30d
Description
#### Description:
With work done in https://github.com/BHoM/BHoM_UI/pull/505 drastically simplifying the workflow for assigning ProjectIds to usage logs, we should aim to compltely remove the [ProjectIDEvent ](https://github.com/BHoM/BHoM_UI/blob/develop/UI_oM/ProjectIDEvent.cs)completely.
To do this, we need to find a way for the [SetProjectId](https://github.com/BHoM/BHoM_UI/blob/develop/UI_Engine/Compute/SetProjectID.cs) method to be able to access UI Name as well as FileId. This could potentially be done by changing this from a basic compute method into a new special component, like GetFileInfo or similar. Then, each UI can make its own implementation of the component and make sure all required data is provided, and then simply call the UpdateProjectId method in added in the PR linked above.
Some downside of adding yet another special case component, so before actioned in the way described above, should investigate if possible to be done another way.
Contributor guide
Research direction
Review UI_oM/ProjectIDEvent.cs, UI_Engine/Compute/SetProjectID.cs, and the workflow changes from PR 505, including UpdateProjectId. Investigate how SetProjectId can access the UI name and FileId, and whether this requires a special component or another approach. Done means ProjectIDEvent is removed while each UI still supplies the required data and usage-log ProjectIds are updated correctly.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- frontend
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100