-
Notifications
You must be signed in to change notification settings - Fork 28
Hotfix #262
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
deeg454
wants to merge
83
commits into
acmutsa:dev
Choose a base branch
from
acmutsa:hotfix
base: dev
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Hotfix #262
Changes from all commits
Commits
Show all changes
83 commits
Select commit
Hold shift + click to select a range
7cde3ba
set up of global styles
MKorolyova 2c45b43
small fixes in global styles
MKorolyova d6a1449
Merge pull request #1 from acmutsa/globals-css-set-up
deeg454 c5f90d2
partner section, partners must be updated later
MKorolyova 6df4bf5
padding editing
MKorolyova 62d36e1
about us component that works on all screen sizes
deeg454 a28d4d0
Merge pull request #2 from acmutsa/partner-section
deeg454 219c336
added some slight animation and fixed ratios
deeg454 0aaeee0
Merge pull request #3 from acmutsa/about-us
deeg454 e16ecbe
initial commit
deeg454 1173c1f
Merge pull request #4 from acmutsa/feat/faq
deeg454 664d482
added motion library
deeg454 4585476
Footer complete
vivtratx 5cf5156
implement map section with dynamic pin positioning, create reusable …
MKorolyova 45a9ab7
Finished hero.tsx, missing animations
William-Chen07 5abf92b
faq rendering fixes
deeg454 46c153a
about section fixes
deeg454 4aff508
initial commit
deeg454 f9327f3
font fixes and styling changes
deeg454 ffc1f8c
Merge pull request #5 from acmutsa/footer
deeg454 bc55416
Merge pull request #10 from acmutsa/map-section
deeg454 7c5595d
Merge pull request #11 from acmutsa/fix/navbar-hero-fixes
deeg454 a1fbce5
footer logo fix
deeg454 567a801
mlh badge fixes
deeg454 a999114
mlh badge for xl screens
deeg454 dd8cce9
fixed ui issues, thread working
Rufat00 fcb42ec
pre animation branch merge changes to ui
deeg454 9344088
hero and faq animations added from fix branch
deeg454 0d2b652
final ui fixes
deeg454 1c750c6
Mlh badge fix
deeg454 2dcd8f3
Merge pull request #14 from acmutsa/ui-fixes
deeg454 3f0ac41
Design fix
MKorolyova 554c167
Merge remote-tracking branch 'origin/string' into assets-changes
Rufat00 5a1e5c6
Pin position and thread size fix for all screen types
MKorolyova edada7b
Deleting Work with as section from main page
MKorolyova b02e251
Change assets from png to webp
MKorolyova b657a85
removing Work with us from main page again
MKorolyova 3b1b1c9
Merge branch 'dev' into assets-changes
Rufat00 6096df6
Merge pull request #18 from acmutsa/assets-changes
Rufat00 d2ae4e2
Organizers section with animation
MKorolyova 699b58c
coderabbitai recomended fix
MKorolyova 5e146fd
fixing env
Rufat00 6bffca2
Merge pull request #20 from acmutsa/organizers-section
Rufat00 17d268b
added cloudflare configuration
Rufat00 65f068a
changed build command
Rufat00 7b86a2f
added necessary flags
Rufat00 be8a4d7
fixed
Rufat00 f0f53c0
bug fix
Rufat00 a99c6c0
some minor fixes
Rufat00 08c0e8f
added wrangler to gitignore
Rufat00 8ec7a54
wrangler config
Rufat00 a65e652
cloudflare preview config fix
MKorolyova 039fe5b
cloudflare upload config fix
MKorolyova 96c9b2e
Merge pull request #22 from acmutsa/cloudflare-config
MKorolyova 8896646
removed redis, nav links are static, registration toggle is through c…
Rufat00 d8ea41e
hackkit config update
MKorolyova 48607a4
Merge branch 'dev' into deleting-redis
Rufat00 590fd94
Update hackkit.config.ts
Rufat00 03fab56
Merge pull request #23 from acmutsa/deleting-redis
Rufat00 8a3e9f8
fixed color theme
Rufat00 8020e8a
lock-file
Rufat00 8883207
FAQ Page
deeg454 1b753ef
faq hover link animation
deeg454 08d1234
smoother fade out transition
deeg454 5e9b63c
final fixes
deeg454 7dc441f
Registration closed message and Registration form fix
MKorolyova 7e9c2e3
Merge pull request #25 from acmutsa/fixing-colors
deeg454 469ebb0
mlh requirements
deeg454 87147de
dashboard
MKorolyova 5315fd8
xl screen fix
MKorolyova 7d38eb9
dashboard navbar
MKorolyova eeaa336
dashboard navbar colour fix
MKorolyova 7ce0e9a
Merge pull request #27 from acmutsa/xl-screen-fix
deeg454 899d52a
event-pass
MKorolyova 2bd163b
Merge pull request #26 from acmutsa/dashboard
deeg454 dcd625d
faq landing animation changes
deeg454 be1df88
pin location changes
deeg454 639e498
smaller screen size pin fix
deeg454 9081737
move one pin
MKorolyova baedf56
Merge branch 'dev' into faq-page
MKorolyova 06329cd
Merge pull request #24 from acmutsa/faq-page
MKorolyova 4a8abb9
added volunteer form link and removed register link
deeg454 efd557c
Update link in Hero component to point to register page
MKorolyova File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -36,4 +36,6 @@ next-env.d.ts | |
|
|
||
| # react-email | ||
| .react-email/ | ||
| certificates | ||
| certificates | ||
| .open-next/ | ||
| .wrangler/ | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| import { defineCloudflareConfig } from "@opennextjs/cloudflare"; | ||
|
|
||
| export default defineCloudflareConfig(); |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
🔒 Security & Privacy | 🟠 Major | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
Repository: acmutsa/HackKit
Length of output: 4536
🏁 Script executed:
Repository: acmutsa/HackKit
Length of output: 4536
🏁 Script executed:
Repository: acmutsa/HackKit
Length of output: 244
🌐 Web query:
OpenNext Cloudflare keep-vars deploy upload wrangler dashboard secrets documentation💡 Result:
To manage secrets and environment variables when deploying OpenNext applications to Cloudflare, follow these recommended practices: Runtime Environment Variables and Secrets For production, you should set environment variables and secrets directly in the Cloudflare dashboard or via Wrangler [1][2]. Secrets are encrypted and cannot be read back after creation [1]. If you are using Workers Builds (CI/CD), ensure you set these in the "Build variables and secrets" section of your project settings in the Cloudflare dashboard, as these are required for the Next.js build process to function correctly [1]. Using --keep-vars When deploying via the OpenNext CLI, you can use the --keep-vars option to prevent existing environment variables on your Cloudflare Worker from being deleted during a deployment. This is typically invoked as follows [1]: opennextjs-cloudflare deploy -- --keep-vars Managing Secrets during Deployment You should avoid hardcoding secrets in your configuration files (e.g., wrangler.toml) [2]. During local development, you can use a.dev.vars file (which should be excluded from version control) [1][3]. For production deployments, Cloudflare Wrangler supports uploading secrets alongside your code. You can use the --secrets-file flag with wrangler deploy or wrangler versions upload to pass secrets securely [4][5]. Best Practices for Development - Local Development: Use.dev.vars or.env files to manage variables locally [1][3][2]. - Accessing Variables: Within your application code, access variables and secrets via the env parameter provided in the getCloudflareContext function or directly from the request context in API routes [3][2]. - Avoid Baking Secrets: Do not include secrets in your source code or configuration files [2]. Always rely on Cloudflare's secure secret storage [1][4]. For consistent and reproducible deployments, it is recommended to use Cloudflare’s Workers Builds system, which integrates directly with your repository, rather than manual command-line uploads [6][7].
Citations:
🌐 Web query:
OpenNext Cloudflare upload keep-vars wrangler versions upload💡 Result:
When using the OpenNext Cloudflare adapter, you can use the --keep-vars flag to prevent your existing environment variables (set in the Cloudflare Dashboard) from being overwritten during deployment [1][2]. If you are using the opennextjs-cloudflare CLI to upload your application, you can pass this flag to the underlying wrangler command by separating it from the CLI arguments with a double dash (--). For example: npx opennextjs-cloudflare upload -- --keep-vars The upload command itself is designed to create a new version of your application on Cloudflare without immediately serving it, which supports gradual deployment workflows [3][4][5]. Internally, this command executes wrangler versions upload [3]. While the CLI handles the complex transformation of your Next.js build for the Workers environment, you can use this --keep-vars option to maintain your dashboard-configured environment variables across updates [1][2]. Note that you should use the opennextjs-cloudflare CLI commands (such as build, deploy, or upload) rather than calling wrangler directly for deployment tasks, as these CLI commands handle necessary processes like R2 cache population and environment configuration automatically [3][4][6].
Citations:
uploadcommand for worker versions opennextjs/opennextjs-cloudflare#559Preserve dashboard vars on deploy and upload.
deployanduploadshould pass--keep-vars; otherwise Wrangler can delete Cloudflare dashboard-managed variables that aren't declared inapps/web/wrangler.toml.Proposed fix
📝 Committable suggestion
🤖 Prompt for AI Agents
Source: MCP tools