issue/162 - Prevent access to calendar and settings page and redirect when not logged in closes #162

[AlexanderTheGrape]

issue/162 - Prevent access to calendar and settings page and redirect when not logged in

• [ ] The pull request is complete according to the following criteria:
  • [x] Acceptance criteria have been met
  • [x] The documentation is kept up-to-date
  • [x] Comprehensive tests (if applicable) have been generated and all pass.
  • [x] The pull request describes the changes that have been made, and enough information is present in the description for any developer to understand what has changed
  • [x] Commits have been squashed (or will be on merge).
  • [x] The branch name is descriptive and follows the pull request title format : {issue/bug...}/(Issue Number) - Name of issue. E.g bug/30-Fix-Project
  • [x] The pull request title is of the following format : {issue/bug...}/(Issue Number) - Name of issue. E.g bug/30-Fix-Project
  • [x] The description uses github syntax to link to the issue. E,g Resolves se701g2/Doto#{Number}
  • [ ] At least two reviewers assigned. One of which must be the assigner of the issue.
  • [x] If there are merge conflicts, run git rebase as opposed to git merge with master.

[AlexanderTheGrape]

There's just one remaining linting issue I haven't been able to change so far. It says "React Hook useEffect has a missing dependency: 'setTheme'.", but when I import setTheme, the import doesn't get used. If any reviewers have a suggestion, please let me know!

[Matteas-Eden]

There's just one remaining linting issue I haven't been able to change so far. It says "React Hook useEffect has a missing dependency: 'setTheme'.", but when I import setTheme, the import doesn't get used. If any reviewers have a suggestion, please let me know!

So I looked into it, and it appears that the code causing the build to fail is:

useEffect(() => {
    const fetchTasks = async () => {
        const tasks = await DotoService.getTasks();
        setTasks(tasks);
    };
    const fetchUserInfo = async () => {
        const userInfo = await DotoService.getUserInfo();
        setTheme(userInfo.themePreference);
    };
    fetchUserInfo();
    fetchTasks();
}, []);

This little code snippet is from doto-frontend/src/components/pages/Calendar/Calendar.js. The reason that the build is failing is because of the empty array after the anonymous function. As in; useEffect(()=>{...}, []) That second argument is causing the build to fail. So, try removing it. It's a little strange because this file wasn't actually changed by your PR, but maybe if this works then your PR can fix it.

[AlexanderTheGrape]

Cheers Matt, I've made that change and works as expected.