Links
Issue : https://github.com/tarasglek/chatcraft.org/issues/626
Pull Request : https://github.com/tarasglek/chatcraft.org/pull/759
Description
This post continues the blog post about Project Release 0.4
Progress
I started to write codes for this issue. Since a lot of changes were needed to be changed in codes, I tried my best to follow the original code style. Especially when I am changing CSS; I spent quite a bit of time on making my new code as similar to original codes as I can, and also I tried to make as less changes in code.
The second thing that was challenging was the design. I've never thought I have a good sense of aesthetics, so I had no idea of what the desgin for the new page should look like. Moreover, in the issue there was no specific guide about what kind of design the maintainers want. It was good in a sense that I have a freedom to make it in my style and apply my creativity. However, it was also a bit confusing.
So I went to the ChatGPT, looked at the settings page, and refered to that design.
ChatGPT Settings page
The third thing that I think about was the namings of the page. I considered among "Data", "Storage", "Data Controls", and "Database". Then, since the most popular and most used AI Chat is ChatGPT, I thought that if we follow the naming styles, it would be beneficial to users and less confusing to the users.
The last thing that I considered was extra fix. There was duplicate declarations of interface; however, this wasn't directly related to the issue. So, I wasn't sure that if I should fix this or make a new issue for this. This was a really easy and simple problem, I thought that there was no need to open a new issue for this. So I just fixed it(removed one of them).
Top comments (0)