compserv / hknweb

The new HKN website (using Django, hopefully at hkn.eecs.berkeley.edu soon)
https://dev-hkn.eecs.berkeley.edu
MIT License
19 stars 108 forks source link

Zayd candidate project files #514

Closed Zaydalii closed 1 year ago

Zaydalii commented 1 year ago

Adding my candidate project files into the repo to create a branch and then submit a pull request.

lgtm-com[bot] commented 1 year ago

This pull request introduces 2 alerts when merging f9dd98f9233426152a48a01bdef77aa447b24fb4 into 2c873ba19fdbb4e936a6910619a90f0b53d01df5 - view on LGTM.com

new alerts:

lo-maxwell commented 1 year ago

Looks like your setup works and the website is running, so that's a good start. I had to change your timezone from PST to UTC in order for it to compile, so you might want to look into what timezone you want to be using. Don't hesitate to ask questions in the compserv channel/dm an officer if you feel stuck.

For your final submission you'll need to move all the poll app files into the actual hknweb website so that we can have everything on one server instance. If you've made more progress than this, it might be easier to keep it as is for now and just move everything over at the end.

Zaydalii commented 1 year ago

Hey,

I’m so sorry, I completely missed this email! Thank you so much for the feedback, I’ll be sure to do that when it comes to the final submission.

Best, Zayd Ali

On Fri, Nov 18, 2022 at 11:42 PM Maxwell Lo @.***> wrote:

Looks like your setup works and the website is running, so that's a good start. I had to change your timezone from PST to UTC in order for it to compile, so you might want to look into what timezone you want to be using. Don't hesitate to ask questions in the compserv channel/dm an officer if you feel stuck.

For your final submission you'll need to move all the poll app files into the actual hknweb website so that we can have everything on one server instance. If you've made more progress than this, it might be easier to keep it as is for now and just move everything over at the end.

— Reply to this email directly, view it on GitHub https://github.com/compserv/hknweb/pull/514#issuecomment-1320826360, or unsubscribe https://github.com/notifications/unsubscribe-auth/A2YCBYGFLR3GSLIZ7B32GM3WJCAFPANCNFSM6AAAAAAR7QTKEU . You are receiving this because you authored the thread.Message ID: @.***>

-- Mir Zayd Ali Electrical Engineering and Computer Science University of California, Berkeley @.***

lo-maxwell commented 1 year ago

Looks good! In an actual merge we would want to delete the zayd_cand_proj/ files since they aren't actually used in the main site. If you want to keep experimenting with django, some optional things to do are: 1) modify the layout of your polls app to match the rest of the hknweb site, 2) make links going to/from the main site to your polls app, 3) mess around with the admin panel and see how data is modified on the main site (might require updating to the most recent repo).

Congrats on finishing your candidate project! Closing this now.

Zaydalii commented 1 year ago

Great! Thank you so much. I’ll delete the folder now, and i’ll try those experiments out in the future! Thank you for all of your help!

Best, Zayd

On Sat, Dec 3, 2022 at 7:00 PM Maxwell Lo @.***> wrote:

Looks good! In an actual merge we would want to delete the zayd_cand_proj/ files since they aren't actually used in the main site. If you want to keep experimenting with django, some optional things to do are: 1) modify the layout of your polls app to match the rest of the hknweb site, 2) make links going to/from the main site to your polls app, 3) mess around with the admin panel and see how data is modified on the main site (might require updating to the most recent repo).

Congrats on finishing your candidate project! Closing this now.

— Reply to this email directly, view it on GitHub https://github.com/compserv/hknweb/pull/514#issuecomment-1336308186, or unsubscribe https://github.com/notifications/unsubscribe-auth/A2YCBYFVWXYTIOQVRSZP4B3WLQCL7ANCNFSM6AAAAAAR7QTKEU . You are receiving this because you authored the thread.Message ID: @.***>

-- Mir Zayd Ali Electrical Engineering and Computer Science University of California, Berkeley @.***