Skip to content

update homepage and event notification for 2026 user survey - #593

Merged
magdmartin merged 2 commits into
masterfrom
2026-user-survey-update
Aug 3, 2026
Merged

update homepage and event notification for 2026 user survey#593
magdmartin merged 2 commits into
masterfrom
2026-user-survey-update

Conversation

@magdmartin

Copy link
Copy Markdown
Member

@netlify

netlify Bot commented Jul 27, 2026

Copy link
Copy Markdown

Deploy Preview for openrefine-website ready!

Name Link
🔨 Latest commit 15b006e
🔍 Latest deploy log https://app.netlify.com/projects/openrefine-website/deploys/6a708fb2aa6a0b00092648b4
😎 Deploy Preview https://deploy-preview-593--openrefine-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@magdmartin
magdmartin requested review from Abbe98 and tfmorris July 28, 2026 12:03
@magdmartin

Copy link
Copy Markdown
Member Author

@tfmorris @Abbe98 I don't want to merge this until a Core Dev Group member approves it since it will notify all OpenRefine users. Thanks

@Abbe98 Abbe98 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The button breaks the layout on medium and small screens but it looks like it has been the case for the last few years.

Better getting this out there sooner than later but lets keep the issue in mind for the future.

@tfmorris tfmorris left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

One small grammar fix for the link label, but otherwise looks OK. I think the banner might work better at the top of the page, rather than buried in the middle, but I'm not a designer.

Not sure how to test the events.json, but we can fix it up later if there's an issue.

From a process point of view, it seems more like something that the project management committee needs to approve rather than the core developers, but I'm fine with reviewing too.

Comment thread src/pages/index.tsx Outdated
Co-authored-by: Tom Morris <tfmorris@gmail.com>
@magdmartin

Copy link
Copy Markdown
Member Author

Thanks for the edit. To test the change, you need to edit this line in your local OpenRefine: https://github.com/OpenRefine/OpenRefine/blob/master/main/webapp/modules/core/scripts/index.js#L162 and replace https://openrefine.org/ with the Netlify URL for this PR, https://deploy-preview-593--openrefine-website.netlify.app/, in this case.

@magdmartin
magdmartin merged commit 5e7e709 into master Aug 3, 2026
4 checks passed
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