Skip to content

Comments

[WIP] Update to Meteor 1.7#11935

Closed
sampaiodiego wants to merge 16 commits intodevelopfrom
update-meteor-1-7
Closed

[WIP] Update to Meteor 1.7#11935
sampaiodiego wants to merge 16 commits intodevelopfrom
update-meteor-1-7

Conversation

@sampaiodiego
Copy link
Member

I'm not happy with the current state of this but I'm opening the pull request to make it public and ask for help =)

I couldn't make @import work on CSS so I decided to move to api.addFiles of every CSS file and because of that, move all files from imports folder to imps (seems api.addFiles doesn't work for files on imports folder 🤔 ).

Also assets needs to fix.

TODO list:

  • Change api.addFiles to @import again (?)
  • Fix assets
  • There is an error happening on synced-cron

* develop: (27 commits)
  Fix stylelint
  [FIX]  Login error message not obvious if user not activated (#11785)
  [NEW] REST endpoints to get moderators from groups and channels (#11909)
  [NEW] User preference for 24- or 12-hour clock (#11169)
  [FIX] Adding scroll bar to read receipts modal (#11919)
  [FIX] Fixing translation on 'yesterday' word when calling timeAgo function (#11946)
  [FIX] Fixing spacement between tags and words on some labels (#12018)
  [FIX] video message recording, issue #11651 (#12031)
  [FIX] Prevent form submission in Files List search (#11999)
  [NEW] REST endpoint to set groups' announcement (#11905)
  [FIX] Re-add the eye-off icon (#12079)
  [NEW] Livechat trigger option to run only once (#12068)
  [IMPROVE] Use eslint-config package (#12044)
  [FIX] Internal error when cross-origin with CORS is disabled (#11953)
  [FIX] Message reaction in GraphQL API (#11967)
  [FIX] Direct messages leaking into logs (#11863)
  Fix the stylelint (#11991)
  [NEW] REST endpoints to create roles and assign roles to users (#11855)
  [FIX] Hidden admin sidenav on embedded layout (#12025)
  [FIX] Update user information not possible by admin if disabled to users (#11955)
  ...

# Conflicts:
#	package-lock.json
#	package.json
WebAppInternals.staticFiles[`/__cordova/assets/${ key }.${ value.cache.extension }`] = value.cache;
// TODO: fix meteor 1.7
// WebAppInternals.staticFiles[`/__cordova/assets/${ key }`] = value.cache;
// WebAppInternals.staticFiles[`/__cordova/assets/${ key }.${ value.cache.extension }`] = value.cache;
Copy link
Member

Choose a reason for hiding this comment

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

@rodrigok do you know what is the impact of this? And how to fix it?

# Conflicts:
#	.meteor/packages
#	package-lock.json
#	package.json
#	packages/rocketchat-livechat/.app/package-lock.json
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11935 October 5, 2018 20:58 Inactive
@rodrigok rodrigok temporarily deployed to rocket-chat-pr-11935 October 16, 2018 15:14 Inactive
@rodrigok rodrigok had a problem deploying to rocket-chat-pr-11935 October 16, 2018 17:27 Failure
@engelgabriel engelgabriel deleted the update-meteor-1-7 branch November 22, 2018 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants