Skip to content

Fix comments scss to avoid compilation errors#10657

Merged
ferblape merged 1 commit intofeature/redesignfrom
fix-comments-scss
Apr 4, 2023
Merged

Fix comments scss to avoid compilation errors#10657
ferblape merged 1 commit intofeature/redesignfrom
fix-comments-scss

Conversation

@entantoencuanto
Copy link
Copy Markdown
Contributor

@entantoencuanto entantoencuanto commented Mar 31, 2023

🎩 What? Why?

This PR contains a quick fix of comments scss avoiding assets compilation errors. With such errors the workflow always got stuck at that point, hence no new deployment was made anymore. Using this common traspilation allows to be working again

@entantoencuanto entantoencuanto added the project: redesign Barcelona City Council contract label Mar 31, 2023
@alecslupu
Copy link
Copy Markdown
Contributor

@entantoencuanto please make sure you do add enough details, so i can review :)

@ferblape ferblape requested a review from a team April 4, 2023 08:29
@ferblape
Copy link
Copy Markdown
Contributor

ferblape commented Apr 4, 2023

Fixed the PR description @alecslupu

@alecslupu
Copy link
Copy Markdown
Contributor

Fixed the PR description @alecslupu

approved :)

@ferblape ferblape merged commit e1ffc49 into feature/redesign Apr 4, 2023
@ferblape ferblape deleted the fix-comments-scss branch April 4, 2023 08:50
entantoencuanto added a commit that referenced this pull request Apr 10, 2023
* feature/redesign:
  Redesign: menu mobile (#10351)
  Fix comments scss to avoid compilation errors (#10657)
  Redesign: Add a new <aside> to layout item (#10620)
  Redesign: author tooltip (#10438)
  Redesign: pending things of the public profile (#10472)
entantoencuanto added a commit that referenced this pull request Apr 10, 2023
* feature/redesign:
  Redesign: menu mobile (#10351)
  Fix comments scss to avoid compilation errors (#10657)
  Redesign: Add a new <aside> to layout item (#10620)
  Redesign: author tooltip (#10438)
  Redesign: pending things of the public profile (#10472)
  Redesign: card L link whole card (#10554)
  strip links from cards, standarize author cell (#10547)
entantoencuanto added a commit that referenced this pull request Apr 12, 2023
…content-blocks

* feature/redesign:
  replace uses of specific margin-bottom for layout margins (#10675)
  Redesign: menu mobile (#10351)
  Fix comments scss to avoid compilation errors (#10657)
  Redesign: Add a new <aside> to layout item (#10620)
entantoencuanto added a commit that referenced this pull request Apr 13, 2023
* feature/redesign:
  replace uses of specific margin-bottom for layout margins (#10675)
  Redesign: menu mobile (#10351)
  Fix comments scss to avoid compilation errors (#10657)
  Redesign: Add a new <aside> to layout item (#10620)
  Redesign: author tooltip (#10438)
  Redesign: pending things of the public profile (#10472)
  Redesign: card L link whole card (#10554)
  strip links from cards, standarize author cell (#10547)
entantoencuanto added a commit that referenced this pull request Apr 28, 2023
…ent-blocks

* feature/redesign: (196 commits)
  Redesign: process & process group cards (#10716)
  Redesign: conference cards (#10502)
  Redesign: pending blogs (#10686)
  Redesign: blog cards (#10685)
  Redesign: filters (#10390)
  replace uses of specific margin-bottom for layout margins (#10675)
  Redesign: menu mobile (#10351)
  Fix comments scss to avoid compilation errors (#10657)
  Redesign: Add a new <aside> to layout item (#10620)
  Redesign: author tooltip (#10438)
  Redesign: pending things of the public profile (#10472)
  Redesign: card L link whole card (#10554)
  strip links from cards, standarize author cell (#10547)
  include a clause to change a data-attr on the fly
  Update tests
  Restore specs
  set different modals foreach comment
  unify report button for futher uses
  use report_button instead of flag_modal
  Unify comments:loaded event
  ...
entantoencuanto added a commit that referenced this pull request May 26, 2023
* feature/redesign: (21 commits)
  Feature/redesign components breadcrumb (#10441)
  Redesign: debates (#10653)
  Redesign: Remove reference to commentable in activity cell (#10816)
  Redesign: spinner (#10848)
  Redesign: omnipresent banner (#10847)
  Add redesign enable ENV variable to control the pipeline (#10610)
  Redesign: progress bar (#10638)
  Redesign: meeting cards (#10722)
  Redesign: pending login (#10699)
  simplify 2col layout (#10819)
  Redesign: processes groups content blocks (#10491)
  Redesign: assemblies content blocks (#10573)
  Redesign: process & process group cards (#10716)
  Redesign: conference cards (#10502)
  Redesign: pending blogs (#10686)
  Redesign: blog cards (#10685)
  Redesign: filters (#10390)
  replace uses of specific margin-bottom for layout margins (#10675)
  Redesign: menu mobile (#10351)
  Fix comments scss to avoid compilation errors (#10657)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

project: redesign Barcelona City Council contract

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants