Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Only show as many E-periods as needed #45

Open
RobertConde opened this issue Sep 11, 2023 · 7 comments
Open

Only show as many E-periods as needed #45

RobertConde opened this issue Sep 11, 2023 · 7 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@RobertConde
Copy link
Collaborator

image

Should not show E1, E2, and E3 because there are no meetings during those periods.

@RobertConde RobertConde added enhancement New feature or request good first issue Good for newcomers labels Sep 11, 2023
@IsRo06
Copy link

IsRo06 commented Jan 29, 2024

Hi! Would like to contribute! Is it okay if I start working on this issue?

@RobertConde
Copy link
Collaborator Author

Hi! Would like to contribute! Is it okay if I start working on this issue?

Yeah, absolutely! I'll be at casual coding today so just let me know if you need any help.

@IsRo06
Copy link

IsRo06 commented Feb 8, 2024

Hey! I tried to start working on this but I'm getting a CORS API key is missing error. I'm guessing I'm missing some sort of .env file! Anyway I can get it?

@RobertConde
Copy link
Collaborator Author

Did you add the .env pinned in the Discord channel to the app/ folder?

@IsRo06
Copy link

IsRo06 commented Feb 8, 2024

I just did! I'm not in the discord so I was a bit lost for a second. All fixed now

@IsRo06
Copy link

IsRo06 commented Feb 11, 2024

Hi! So I've working on this issue fora couple of days now and haven't been able to get it work properly. While I was able to get it to show no E periods at all as soon as there is a class in one of them things start getting a bit crazy (It no longer generates more schedules and the positionin gets weird)

I feel that I've been stucked bc its been really hard for me to understand how the code is written and what triggers what.

image
Attatched is an image of where I'm stucked rn.

Bc of this i'll be taking myself off of the issue so that someone else could work on it. Sorry :(

@IsRo06 IsRo06 removed their assignment Feb 11, 2024
@Nihar3430
Copy link

I'll claim this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants