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.

Sign up for GitHub

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

Jump to bottom

Content Library #6

Merged
merged 14 commits into from
Nov 8, 2023
Merged

Content Library #6

merged 14 commits into from
Nov 8, 2023

Conversation

Bahugunajii
Copy link
Member

@Bahugunajii Bahugunajii commented Oct 26, 2023 •
edited
Loading

Description

  • Added Content Library page

Checklist

  • Self testing

Steps to test:

  • Set NEXT_PUBLIC_AF_DB_SERVICE_URL to staging-db URL in .env
  • Set NEXT_PUBLIC_AF_DB_SERVICE_BEARER_TOKEN in .env

@Bahugunajii Bahugunajii changed the base branch from main to feature/reports-page October 26, 2023 17:01
@Bahugunajii Bahugunajii changed the title Feature/home page Content Library Oct 29, 2023
@Bahugunajii Bahugunajii self-assigned this Oct 30, 2023
suryabulusu
suryabulusu reviewed Oct 30, 2023
View reviewed changes
Copy link

@suryabulusu suryabulusu left a comment •
edited
Loading

Choose a reason for hiding this comment

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

looks good..left comments.

maybe in next PR, can you add SVGs instead of PNGs?

@Bahugunajii Bahugunajii changed the base branch from feature/reports-page to main November 8, 2023 06:50
@Bahugunajii Bahugunajii merged commit 5385fe6 into main Nov 8, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@suryabulusu suryabulusu suryabulusu approved these changes

Assignees

@Bahugunajii Bahugunajii

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Bahugunajii @suryabulusu