BHoM / BHoM/BHoM_UI

Remove ProjectIDEvent

Open
#506 0 comments 0 reactions 0 assignees View on GitHub
type:feature
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.