feat(tour) designed tour of the board UI, created tour context & card step component. - #81
feat(tour) designed tour of the board UI, created tour context & card step component.#81moffatethan wants to merge 5 commits into
Conversation
moffatethan
changed the title
|
Merge conflicts will be addressed after approval to merge. |
| if (!hasSeenTour) { | ||
| setIsTourMode(true); | ||
| } | ||
| }); |
There was a problem hiding this comment.
I forgot to add in the empty dependency array so it runs on just render as opposed to right now which is running on render then on every update.
| const [tourSteps] = useState<ReactourStep[]>(steps || []); | ||
|
|
||
| useEffect(() => { | ||
| const hasSeenTour = localStorage.getItem('hasSeenTour'); |
There was a problem hiding this comment.
For this project, just utilizing localStorage works however how it works currently is pretty static to only the tour of the board. For extending this and working with showcasing other features (i.e., teams UI), the implementation would have to be touched up. However, the TourProvider is flexible with updating the steps so just some tweaking for a better tour system.
| ); | ||
| }; | ||
|
|
||
| export const steps: ReactourStep[] = [ |
There was a problem hiding this comment.
There is no step for adding a column, I will add that in as it's a pretty hidden feature and probably best to highlight it.
|
@moffatethan Large scale bug that exists likely due to our dashboard redirecting; check out this video here and here. Wasn't able to test unfortunately due to these crashes 😞 |
2 participants
What this PR does (required):
Screenshots / Videos (required):
Any information needed to test this feature (required):
hasSeenTourproperty in LocalStorage.Any issues with the current functionality (optional):