-
-
Notifications
You must be signed in to change notification settings - Fork 401
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
Documentation enhancement #3703
base: master
Are you sure you want to change the base?
Conversation
…12/community-platform into documentation-enhancement
I just realized I created this branch out of the "move notification logic from discussion store to firebase functions" branch. |
FAST_REFRESH: false, | ||
} | ||
|
||
exports.cra = cra |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this still needed? we shouldn't use cra or webpack anymore
@goratt12 Should we just close this? What's needed to get it merged in? |
PR Checklist
PR Type
Description
What this PR does
Git Issues
Closes #
Screenshots/Videos
If useful, provide screenshot or capture to highlight main changes
What happens next?
Thanks for the contribution! We try to make sure all PRs are reviewed ahead of our monthly maintainers call (first Monday of the month)
If the PR is working as intended it'll be merged and included in the next platform release, if not changes will be requested and re-reviewed once updated.
If you need more immediate feedback you can try reaching out on Discord in the Community Platform
development
channel.