-
Notifications
You must be signed in to change notification settings - Fork 121
Update documentation branding: change copyright, links, and config se… #904
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
base: enext
Are you sure you want to change the base?
Conversation
* Fix: Export check-in list to PDF with HTML questions * Improve per sourcery-ai comment
* Improve error message when exporting orders failed * Improve exception catching
* fix: Moved email-content to Templates tab under Mails * implemented sourcery-ai suggestions * replace deprecated with modern syntax
* Update views.py * Update views.py * Update views.py here updated changes as asked by Auswin
…a#752) * chore: rename the mails page heading to match the sidebar menu item * implemented sourcery-ai suggestions
* Added text tab in eventyay-common control * Added a success message * Correct logic from previous commit
* Refactor: Consolidate General Event Settings to eventyay_common * formatting changes * updates * Fix issue with saving event settings form in ticket and common component * formatting fixes * removed event>location autosuggestions
* Fix: the reviews * Fix: removed the @media fadeIn * fix: additional adjustments for fropdoem menu * Update main.scss for the 800*1280 working of the submenu * Update utils.js for the submenu fix in the 800*1280 * Update main.scss, with the z-index vales * Update main.scss. updated the z-index to lower values * Update main.scss. updated the z index to 10s and 20s --------- Co-authored-by: Your Name <[email protected]>
* Fix: Missing menu in Ticket Buying page * Prevent attaching event handler many times
…ia#799) * fix: Tickets General settings crash due to unused kwarg value * fix: Tickets General settings crash due to unused kwarg value
* Render "Join Video" link in email * Fix usage of %s format in logging * Prevent circular import of Order by dropping the Typing --------- Co-authored-by: Norbert Preining <[email protected]>
…#871) * Back to use upstream django-allauth * Set User-Agent for HTTP client when accessing Wikimedia API
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThis PR rebrands the documentation by updating project metadata in the Sphinx config, revising landing page links and text to reflect the switch from Twitter to X, and adjusting footer styling for consistent branding. Class diagram for updated Sphinx documentation configclassDiagram
class conf_py {
- project: str
- copyright: str
+ master_doc: str
}
conf_py : project = "eventyay"
conf_py : copyright = "2025 Apache 2.0 License by contributors"
File-Level Changes
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
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.
Hey there - I've reviewed your changes - here's some feedback:
- Fix the typo in index.html (“gateway tso” → “gateway to”).
- Run a global search for “pretix” and update any remaining references in docs and theme directory names.
- Consider renaming the “_themes/pretix_theme” folder (or adding an Eventyay logo/favicon) to fully reflect the new branding.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Fix the typo in index.html (“gateway tso” → “gateway to”).
- Run a global search for “pretix” and update any remaining references in docs and theme directory names.
- Consider renaming the “_themes/pretix_theme” folder (or adding an Eventyay logo/favicon) to fully reflect the new branding.
## Individual Comments
### Comment 1
<location> `doc/_templates/index.html:112` </location>
<code_context>
<strong>Project website</strong>
</a>
- <p>eventyay.com is your gateway to the eventyay project, offering premier commercial hosting services all in one place.</p>
+ <p>eventyay.com is your gateway tso the eventyay project, offering premier commercial hosting services all in one place.</p>
</div>
</div>
</code_context>
<issue_to_address>
**issue (typo):** Typo in 'tso' should be 'to'.
```suggestion
<p>eventyay.com is your gateway to the eventyay project, offering premier commercial hosting services all in one place.</p>
```
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
doc/_templates/index.html
Outdated
| <strong>Project website</strong> | ||
| </a> | ||
| <p>eventyay.com is your gateway to the eventyay project, offering premier commercial hosting services all in one place.</p> | ||
| <p>eventyay.com is your gateway tso the eventyay project, offering premier commercial hosting services all in one place.</p> |
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.
issue (typo): Typo in 'tso' should be 'to'.
| <p>eventyay.com is your gateway tso the eventyay project, offering premier commercial hosting services all in one place.</p> | |
| <p>eventyay.com is your gateway to the eventyay project, offering premier commercial hosting services all in one place.</p> |
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.
Hi, thanks for working on the project. The next version of the system is being developed in the "enext" branch. All PRs should be made against this branch. Thanks!
|
I've changed the base branch to enext as requested, but there are now merge conflicts showing in multiple files. Since my PR only modified documentation files (index.html, conf.py, and copyright notices), I'm not sure how to proceed with these conflicts. Could you advise on the best way to resolve this? Thanks! |
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.
hi @Anaa1101 you can copy the files you have made changes to to a new place and then use git fetch origin and then use git reset --hard origin/enext on the branch this will reset your branch to enext thereafter, you can choose to cherry-pick(this may or may not work properly) or commit your changes after making commits on the right branch a force push shall do the trick.
Also, please move that style change to the stylesheet file. If you need the guide on how to set up the enext branch, you will find the setup guide here: https://github.com/fossasia/eventyay-tickets/tree/enext
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.
This change should go inside pretix.css; it should be inside static/css within the same directory.
This PR addresses issue #514 by making the following changes:
Completed:
Questions for maintainers:
Summary by Sourcery
Update documentation and configuration to reflect eventyay branding, replacing pretix references, adjusting social media links and legal notices
Enhancements:
Documentation: