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.

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

Update web3version branch to latest forem/main #9

Open
wants to merge 1,160 commits into
base: web3dev-version
Choose a base branch
from

Conversation

cancelei
Copy link
Collaborator

@cancelei cancelei commented Nov 7, 2024

What type of PR is this? (check all applicable)

  • [ x ] Update
  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

Related Tickets & Documents

N/A

  • Related Issue #
  • Closes #
    N/A

QA Instructions, Screenshots, Recordings

Please replace this line with instructions on how to test your changes, a note
on the devices and browsers this has been tested on, as well as any relevant
images for UI changes.

UI accessibility concerns?

If your PR includes UI changes, please replace this line with details on how
accessibility is impacted and tested. For more info, check out the
Forem Accessibility Docs.

Added/updated tests?

  • [ X] Yes
  • No, and this is why: please replace this line with details on why tests
    have not been included
  • I need help with writing tests

[Forem core team only] How will this change be communicated?

Will this PR introduce a change that impacts Forem members or creators, the
development process, or any of our internal teams? If so, please note how you
will share this change with the people who need to know about it.

  • I've updated the Developer Docs or
    Storybook (for Crayons components)
  • This PR changes the Forem platform and our documentation needs to be
    updated. I have filled out the
    Changes Requested
    issue template so Community Success can help update the Admin Docs
    appropriately.
  • I've updated the README or added inline documentation
  • I've added an entry to
    CHANGELOG.md
  • I will share this change in a Changelog
    or in a forem.dev post
  • I will share this change internally with the appropriate teams
  • I'm not sure how best to communicate this change and need help
  • This change does not need to be communicated, and this is why not: please
    replace this line with details on why this change doesn't need to be
    shared

[optional] Are there any post deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

alt_text

Mac Siri and others added 30 commits April 24, 2024 15:14
* Create AlgoliaSearchable::SearchablePodcastEpisode

* Refactor

* Remove pre-mature methods
* Increment Algolia search frontend

* Increment Algolia search frontend

* Update @cypress/code-coverage to version 3.12.36 (#20887)

Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>

* Update ahoy.js to version 0.4.4 (#20889)

Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>

* Add AlgoliaSearchable to Tag (#20888)

* Add AlgoliaSearch::SearchableTag

* Refactor

* Update app/models/concerns/algolia_searchable/searchable_tag.rb

* Add AlgoliaSearchable to Podcast (#20890)

* Create AlgoliaSearchable::SearchablePodcastEpisode

* Refactor

* Remove pre-mature methods

* Add sort replicas

---------

Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
Co-authored-by: Mac Siri <[email protected]>
Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
Co-authored-by: Mac Siri <[email protected]>
* Try create account flaky spec

* Revert "Try create account flaky spec"

This reverts commit ad2eb73.

* Try with basic sleep

* Refactor
Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
* Add AlgoliaSearchable to Article

* Remove comments

* Fix spec
* Add AlgoliaSearchable to Comment

* Improve test

* Apply suggestions from code review

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Refinement

* Fix broken spec

* Apply suggestions from code review

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* Add search branding option

* Change search test
Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
* fix: copy to clipboard message

* chore: deletion of unused imports
* changed the twitter icon to X icon in assets/images/twitter.svg

* Changed the twitter icon to X in assets/images/twitter.svg
* Add field test for Digest article list design

* Add check for cached user
* Include cached user in select

* Add tags
* Fix Twitter/X consistency

* Adjust tests

* Adjust test regex
* Fix support for adding social image to the pages api

* refactor£

* test for the test god
* Add proper frontend for Algolia search

* Update app/models/concerns/algolia_searchable/searchable_comment.rb

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update app/models/concerns/algolia_searchable/searchable_article.rb

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Fix formatting

* Fix: copy to clipboard message (#20898)

* fix: copy to clipboard message

* chore: deletion of unused imports

* Pick winner in header field test (#20911)

* Updated the twitter icon to X icon - solved icon issue #20356  (#20910)

* changed the twitter icon to X icon in assets/images/twitter.svg

* Changed the twitter icon to X in assets/images/twitter.svg

* Add field test for Digest article list design (#20912)

* Add field test for Digest article list design

* Add check for cached user

* Include cached user in select (#20913)

* Include cached user in select

* Add tags

* update style for new follower email (#20908)

* Fix #19037 (#20917)

* Add alternate returns for html

* Remove console log

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Iasmim Pereira <[email protected]>
Co-authored-by: Darshan Padia <[email protected]>
Co-authored-by: Philip How <[email protected]>
* Update request_store to version 1.6.0

* Add temp fix to spec

---------

Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
Co-authored-by: Mac Siri <[email protected]>
Co-authored-by: Mac Siri <[email protected]>
Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
* new badge display

* fix cypress

* fix cypress

* Fit grid to exact number

* change point at which badges shown on profile header to 7

---------

Co-authored-by: Ben Halpern <[email protected]>
benhalpern and others added 30 commits November 12, 2024 20:03
* Fix legacy pagination issues

* Adjust display logic
* Add following feed

* Make adjustments to get working

* Valid sql statement

* Adjust test

* Adjust snapshot

* Adjust system test

* Proper feed call
* Simplify following tab

* Simplify following tab
* Adjust following query to fetch all

* Missing sidebar

* Fix stories url
* Tweaks to magic link underlying functionality

* Send reset link
* Finalize magic link

* Clean up specs
* Remove edge cache safety check

* Comment out play/pause

* Minor rails version upgrade

* Update rexml
* Boost posts via quickie

* Adjust display logic

* Reverse after using

* Reverse after using

* Link share boost i18n

* Don't include embed in quickie
* Style tweaks

* Style tweaks for boost
* Fix newsletter unsubscribe

* Cache article share content
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.