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

Refresh DB section update #280

Closed
wants to merge 1 commit into from
Closed

Refresh DB section update #280

wants to merge 1 commit into from

Conversation

punyakrit
Copy link
Contributor

It would show only if no course is visible if courses are visible it wont show in main page
Screenshot 2024-03-27 at 11 59 26 AM

@devsargam
Copy link
Collaborator

Actually bhai that's expected behaviour 🫡

@punyakrit
Copy link
Contributor Author

But still why show a unwanted pop up of text in main course page only show if course list are not fetched from db

@devsargam
Copy link
Collaborator

Long story short
It's a hack for bypassing cache. I didn't see how it could be done the other way so it's gonna remain there for now

@punyakrit
Copy link
Contributor Author

lets see if it is a valid contribution @hkirat

@hkirat
Copy link
Contributor

hkirat commented Apr 1, 2024

@devsargam is right
It is for busting the cache

@hkirat hkirat closed this Apr 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@punyakrit @devsargam @hkirat