Skip to content

Add opportunity grants #8

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

nanjekyejoannah
Copy link
Contributor

Updated diversity templates

The backend needs an update to flag fin_aid flags to active for the information to show

@Immy-delish or @HassanBahati can help before deploy.

I dont know if we moved our css here from the old website, I can stop referencing the old styles if so.

Copy link
Member

@HassanBahati HassanBahati left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I dont know if we moved our css here from the old website, I can stop referencing the old styles if so.

Thanks @nanjekyejoannah for opening this pr.

Actually, this codebase uses a different Css framework so moving styles from the prev wouldnt be compatible. Currently here we are having bootstrap versus tailwindcss which was previously used.

Given a lot of things here are already with bootstrap, would be smoother to continue with it.

So ideally, in this codebase, classNames would become class. ClassName was in the previous since we were using react is nolonger the case.

The contents of the now class attributes would as well need to be converted to boostrap-like styles. From a quick google search, i could see some tailwindcss to bootstrap converters, though, would need to try them and affirm the effectiveness of those converters which included docuwriter, gitloop. Another quick work around that I'd go with is to use LLMs, the likes of chatgpt, claude to make the conversion of styles from tailwindcss to bootstrap

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's standardize by using the class attribute not the className attribute.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants