Stability & UX issues - Githubissues

osoc22 / project-idlab

We have partnered up with IDLab and Imec to showcase how "taking back control of your personal data" can be achieved using linked data and Solid pods by creating a smart calendar application.

https://osoc22.github.io/project-idlab/

MIT License

4 stars 0 forks source link

Stability & UX issues #78

Closed Denperidge closed 2 years ago

Denperidge commented 2 years ago

[x] Remove test Events (or move them to a tester() like function
[x] Weather notify is not saved in Solid
[x] Activity type is not saved properly in Solid
[x] The possible dates/times (in planning view) are (if I'm correct) deprecated? Its a bit unclear, but either way its not saved in the solidpod. It should either A) be properly saved in the solidpod | or B) the UI should be changed
[x] Either an event should be able to be made without date & time, or the creation process should be blocked if they're omitted

Matei9721 commented 2 years ago

@Denperidge Weather notify meaning on what weather to be notified?

Denperidge commented 2 years ago

@Denperidge Weather notify meaning on what weather to be notified?

I think so! its a set of WeatherTypes (which are strings)

Denperidge commented 2 years ago

remove test Events is in https://github.com/osoc22/project-idlab/pull/77/commits/d1df964558e59eefc038f2d7c71bd700a793abca !

Denperidge commented 2 years ago

Activity type is now saved under Keywords since commit https://github.com/osoc22/project-idlab/pull/77/commits/cacc7bff2a868c95c81ed72137012b0b312909f5

Denperidge commented 2 years ago

Thanks to #82 notifyOnWeather will be saved!

Denperidge commented 2 years ago

After discussing point 4 with @yustarandomname and the rest of the team: the planning view will be kept, but should have a notification of "hey this is proof of concept even within the proof of concept bc this only stores client side"

Denperidge commented 2 years ago

Point 4 is fixed by https://github.com/osoc22/project-idlab/commit/acfbd6ecc3fac2552d6390b38575745346b949c2 which is totally not a commit I pushed straight into master

Denperidge commented 2 years ago

Point 5 fixed by #91!